Skip to content

Conversation

AndrewMbugua
Copy link

Changing from $make run to $bazel build after encountering this issue As part of the Make to Bazel switch-over of the AGW this Makefile will be deprecated soon" message.

Summary

On running $ make run I got the message: MAIN_FEATURE agw_of OAI_FLAGS -DS6A_OVER_GRPC=True -DEMBEDDED_SGW=True Warning: As part of the Make to Bazel switch-over of the AGW this Makefile will be deprecated soon, see https://github.com/magma/magma/issues/14971. Do not add any new Make targets for the AGW! All new targets should be integrated with the Bazel build system, see https://magma.github.io/magma/docs/next/bazel/agw_with_bazel.

and a subsequent build failure.

Test Plan

I initially encountered permission issues so I first did the following(Not recommended for everyone):
$ sudo mkdir -p /var/cache/bazel-cache
$ sudo chown -R vagrant:vagrant /var/cache/bazel-cache

Also do the same for the repository cache:

$ sudo mkdir -p /var/cache/bazel-cache-repo
$ sudo chown -R vagrant:vagrant /var/cache/bazel-cache-repo
$ cd /home/vagrant/magma/lte/gateway
$ bazel build

Result INFO: Build completed successfully, 1 total action

Additional Information

  • This change is backwards-breaking

Security Considerations

Changing from $make run to $bazel build after encountering this issue where "As part of the Make to Bazel switch-over of the AGW this Makefile will be deprecated soon" message.
@lucaaamaral
Copy link
Collaborator

Have you explored the lte/gateway/deploy/agw_install_docker.sh? How would you feel in documenting this path instead? Also, you have changed a versioned doc, which needs to be kept static.

If I may recommend: do create a file in readmes/deploy/agw_docker.md for documenting the AGW installation on docker.
This integrates well with the current plan (#32) of reorganizing the documentation structure.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants