Skip to content
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

Consolidate edge-internal and edx-internal and possibly sandbox-internal #859

Open
2 tasks
robrap opened this issue Nov 20, 2024 · 0 comments
Open
2 tasks

Comments

@robrap
Copy link
Contributor

robrap commented Nov 20, 2024

Context

We learned that the main reason for separate repos were to allow different levels of access to each. This was when these repos contained secrets, that have since been encrypted. This opens the possibility of consolidating.

Additionally, the separate repositories some makes it harder to implement, or remember to implement, similar changes across all environments.

Lastly, containerization will likely change how much code in these repos remains relevant, and would still need to be consolidated.

Acceptance Criteria

  • Consolidate edge-internal into edx-internal
  • Consolidate sandbox-internal into edx-internal (or decide against)

Additional Notes

  • Since the work will change post edxapp containerization, and is likely a lower priority, we shouldn't review this until edxapp containerization is complete. (Possibly other containerization as well.)
  • It is unclear how much work is involved for sandbox-internal, and its benefit, so there should be some discovery on that front.
  • It may make sense to consolidate (and possibly ticket) parts of the repo at time.
@robrap robrap added this to Arch-BOM Nov 20, 2024
@robrap robrap converted this from a draft issue Nov 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Backlog
Development

No branches or pull requests

1 participant