-
Notifications
You must be signed in to change notification settings - Fork 150
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
Add source control support #1228
Comments
The basic requirements for most researches in my experience are just an ability to have remote repositories hosted within the workspace so that small scale code sharing can occur but more importantly version history is preserved. As described the requirements are really just to be able to create repositories and push code to main and other branches. Minimal implementation will make any information governance questions easier and have lower operating costs. |
I suggest a workspace service making use of the work done in the Gitea shared service bundle (part of #1177 ) |
An alternate option was suggested, where we have a proxy in front of a shared Git provision restricting access to each Git organization by workspace IP range. I think this might be something to look at longer term. |
Being able to have git organisations per workspace makes great sense and then being able to have the Git provision as part of the TRE Core would mitigate the cost per workspace issue an additional instance in each workspace brings. That seems like a good longer term solution. |
Just repeating a discussion here from SDF TRE: We'd ideally want
|
Closing this as source control complete. Acknowledge secure ML Ops needs a solution. Will create a separate issue for GitLab, or might be covered as part of #1502 albeit with licensing implications. |
Describe the solution you'd like
As a researcher I want to be able to use source control within a workspace service
Additional context
Ability to create repositories
Create branches
Commit code
Add git source control to workspace service #1299
The text was updated successfully, but these errors were encountered: