Skip to content

Pre-1.9 merge of magma-deployer #6

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

Merged
merged 33 commits into from
Apr 28, 2025
Merged

Pre-1.9 merge of magma-deployer #6

merged 33 commits into from
Apr 28, 2025

Conversation

jblakley
Copy link
Collaborator

Summary

Magma-deployer was created at 1.8 to deploy containerized AGWs. The main branch has not been updated since then. This PR pulls in pre-1.9 revisions developed for the 5G deployment at Carnegie Mellon University. It restructures the deployer to include both AGW and Orc8r deployment and updates the AGW deployer for pre-1.9 deployments. Also, includes a number of clean up items.

This PR brings the main branch up to a level that worked at pre-release levels of 1.9. It will need to be updated to get it to the release 1.9 release level. The primary updates will need to be around the artifact (e.g., container) retrievals from jfrog with the new names and keys.

Test Plan

This PR was tested for both AGW and Orc8r during the 5G deployment at CMU. It worked for that specific deployment but may have some environment specific issues. See this article.

Additional Information

This change is backwards breaking.
It will need additional updates for production level 1.9.

Copy link

@lucasgonze lucasgonze left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Numerous substantial improvements in a valuable project that is not mission critical - no brainer. I'm approving without reading or running the code, for the sake of clearing bureaucracy.

@jblakley jblakley merged commit 9e9118c into magma:main Apr 28, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants