-
Notifications
You must be signed in to change notification settings - Fork 592
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat(agw): add architecture value to helm charts to support ARM deploy… #14863
Conversation
Thanks for opening a PR! 💯
Howto
More infoPlease take a moment to read through the Magma project's
If this is your first Magma PR, also consider reading
|
✔️ The Semantic PR check ended with status success. See instructions on formatting your commit and pull request titles. |
…yment Signed-off-by: Jim Blakley <jblake1@andrew.cmu.edu>
896ecf1
to
988e1b9
Compare
Workflows should now be running on the PR - some workflows require manual approval to run for first time contributors. Thank you for contributing. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
…yment (magma#14863) Signed-off-by: Jim Blakley <jblake1@andrew.cmu.edu>
Signed-off-by: Jim Blakley jblake1@andrew.cmu.edu
Summary
Updated agwc-helm-charts to add a value "arch" and modified the templates to use this value on image pull. This enables helm to pull the ARM images rather than the x86 images on a
helm install
. README was updated accordingly.The code submitted also takes out the installation of the GW_CHALLENGE_KEY. This was not working correctly and was done manually as described here: https://docs.magmaindia.org/deploy/agw/install-agw-helm.html.
Test Plan
Tested on an ARM64 Ampere server. AGW has been deployed in operation in our network for over a week with no issues. (including eNB and connected UE). Tested
helm install --dry-run
to verify that x86 images urls remain correct.Additional Information