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

Switch edx/configuration to a private repo #782

Open
robrap opened this issue Aug 27, 2024 · 2 comments
Open

Switch edx/configuration to a private repo #782

robrap opened this issue Aug 27, 2024 · 2 comments

Comments

@robrap
Copy link
Contributor

robrap commented Aug 27, 2024

Do we want the edx/configuration repo to be private copy, rather than a public fork?

Arguments for:

  • The repo contains 2U-specific configuration. Anything that is business-private, but not a secret, needs extra work to make it a secret.

Arguments against:

  • This would require additional work to have jobs use tokens to reach the now-private repo, rather than having it be publicly accessible.
  • For some repos and some types of github actions, it might be more expensive to run from a private repo.
  • Not everyone agrees on whether we can legally switch from public to private based on licensing. Most agree that this particular repo would probably be ok.
  • Regarding this repo specifically, we are trying to move off of it more and more as we complete containerization work.

In general, it seems like public forks are the simplest way to go, and should be the solution unless there is compelling reason to do otherwise.

@robrap robrap added this to Arch-BOM Aug 27, 2024
@robrap robrap converted this from a draft issue Aug 27, 2024
@robrap robrap moved this to Backlog in Arch-BOM Aug 27, 2024
@robrap
Copy link
Contributor Author

robrap commented Aug 27, 2024

[inform]

@robrap robrap removed the status in Arch-BOM Aug 27, 2024
@robrap robrap moved this to Backlog in Arch-BOM Sep 6, 2024
@jristau1984
Copy link

Check this out again in a month or so, see if these scenarios have come up again. If not, this can be closed.

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

2 participants