Skip to content

Move blocked repository configuration into the database and build admin UI to work with it #11030

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

Closed
8 tasks done
andrew-farries opened this issue Jun 30, 2022 · 2 comments · Fixed by #11462
Closed
8 tasks done

Comments

@andrew-farries
Copy link
Contributor

andrew-farries commented Jun 30, 2022

Gitpod allows for repositories to be blocked, so that workspaces can't be started with those contexts. Support for blocking repositories was added to components/server in #9762 where the implementation used server config to provide the list of blocked repos.

We'd like to move blocked repository configuration out of server config and into the application database for a few reasons:

  • We can more easily build blocked repo configuration into the Gitpod admin UI (and/or gpctl?)
  • Having this configurable in the admin UI makes on-call easier than having to change server config at runtime with kubectl.
  • The ability to block repositories may be useful for self hosted users.

Implementation

We don't need to use a feature flag for this; we can support blocked repository config in the database and in server config at the same time. We can then move the repositories that reside in server config into the database and delete the former.

Tasks

@geropl
Copy link
Member

geropl commented Jul 18, 2022

@andrew-farries Do you mind if I take over here during my on-call week? ❓

@andrew-farries
Copy link
Contributor Author

Please do 😄

@geropl geropl self-assigned this Jul 18, 2022
@geropl geropl moved this to In Progress in 🍎 WebApp Team Jul 18, 2022
Repository owner moved this from In Progress to Done in 🍎 WebApp Team Jul 20, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

2 participants