- Notifications
You must be signed in to change notification settings - Fork673
feat(job_token_scope): support Groups in job token allowlist API#2816
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
feat(job_token_scope): support Groups in job token allowlist API#2816
Uh oh!
There was an error while loading.Please reload this page.
Conversation
NOTE: Group allowlist functionality is due in I will develop tests using the |
11ac87a
toe3576be
Compare5f27abe
to5aa3701
CompareReady for Review For merge in will need to mark as XFail or have a feature flag based on if 16.10+ is active or no? |
codecovbot commentedMar 5, 2024 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@## main #2816 +/- ##==========================================- Coverage 96.52% 96.50% -0.03%========================================== Files 90 90 Lines 5872 5889 +17 ==========================================+ Hits 5668 5683 +15- Misses 204 206 +2
Flags with carried forward coverage won't be shown.Click here to find out more.
|
9d5b099
to9386847
Comparea815a6f
tod2cc51e
Compare8771e7d
to8f12f86
Compare@nejch built on the work you started and some of the discussions we had a few months back, this adds support for managing the CI Job Token via python-gitlab. |
Uh oh!
There was an error while loading.Please reload this page.
0481c52
to2bbe2ba
CompareHi@nejch ,@JohnVillalovos ,@max-wittig ,@bufferoverflow Myself and@SachinKSingh28 among others will be working on a project to migrate from Gitlab-SAAS to Dedicate, alongside our work on GitLabForm project we will be adding in Enterprise feature sets to python-gitlab. Would there be a good way to discuss a ways of working that is useful for the demands on your time, and if it would be helpful for us to also take on Maintainership of the project? As it's a key part of our infrastructure here at the DWP (a UK Gov public sector department) |
Nothing personal but after what happened to XZ project [1]. Having people asking to become maintainers is looked warily by me. https://news.ycombinator.com/item?id=39865810 But I would be up for a video-conference if@nejch would like to do it. Doing a hackathon or something along those lines could be fun. |
Uh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
Overall this is looking pretty good to me. My suggestions are mostly minor. |
TimKnight-DWP commentedMay 8, 2024 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
@JohnVillalovos -> Oh absolutely! We work in Security here, and having seen what happen with XZ is why I've not wanted to be too pushy 👍 We would be up for having a video-conference call etc. We're on BST |
2bbe2ba
to4f5cfed
CompareUpdated PR |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
This is looking good to me. Left a couple of comments about trying to get the code coverage test to pass.
Hopefully@nejch can do a review on this too.
Uh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
@JohnVillalovos updated once more, I certainly intended to have already covered those paths, but clearly hadn't so done now 👍 |
e1d5f8d
toee826ad
CompareSigned-off-by: Tim Knight <tim.knight1@engineering.digital.dwp.gov.uk>
Signed-off-by: Tim Knight <tim.knight1@engineering.digital.dwp.gov.uk>
Signed-off-by: Tim Knight <tim.knight1@engineering.digital.dwp.gov.uk>
Signed-off-by: Tim Knight <tim.knight1@engineering.digital.dwp.gov.uk>
Signed-off-by: Tim Knight <tim.knight1@engineering.digital.dwp.gov.uk>
Signed-off-by: Tim Knight <tim.knight1@engineering.digital.dwp.gov.uk>
Signed-off-by: Tim Knight <tim.knight1@engineering.digital.dwp.gov.uk>
ee826ad
to4aab77f
CompareThanks for the merge@JohnVillalovos If you wanted to have a chat, we can talk through what we're doing with GitLab on our end and how that'll interface with python-gitlab. Hopefully this calendly link should work:https://calendly.com/tim-knight1/30min |
Meeting sounds great to me, but I would like to wait for@nejch to join. I'm assuming that he is on vacation or something as I haven't heard from him for a bit. |
Builds ontop of:#2767 and#2790
Closes#2762
Adds support for adding Group to job_token_allowlist:https://docs.gitlab.com/ee/api/project_job_token_scopes.html#add-a-group-to-a-cicd-job-token-allowlist