Document scripts/enroll-users.py for Fedora CI
#1085
Conversation
|
Build succeeded. ✔️ pre-commit SUCCESS in 1m 02s |
There was a problem hiding this comment.
The new script looks good to me.
I am only wondering whether we should also keep the old "bulk" script.
I don't know how we are supposed to deal with orphaned packages. Should we run the bulk script once a year (or more) to get rid of un-maintained packages? In that case the old script probably needs some adjustments.
Sorry I posted the comment in the wrong PR
Convert the zuul migration script into enrollment script This converts the old script used to migrate the zuul users to instead do what zuul-config-generator did. Update or write new documentation in packit/packit.dev. packit/packit.dev#1085 Merge after #672 RELEASE NOTES BEGIN Fedora-CI users can be enrolled by running the script scripts/enroll-users.py. This replaces the script zuul-config-generator that was used in the Zuul CI. RELEASE NOTES END Reviewed-by: Maja Massarini Reviewed-by: Laura Barcziová Reviewed-by: Cristian Le <github@lecris.me>
|
Build succeeded. ✔️ pre-commit SUCCESS in 1m 04s |
|
Build succeeded (gate pipeline). ✔️ pre-commit SUCCESS in 1m 07s |
|
Pull request merge failed: 2 of 2 required status checks are expected. |
|
@LecrisUT would you rebase this please? |
Co-authored-by: Laura Barcziová <49026743+lbarcziova@users.noreply.github.com>
LecrisUT
force-pushed
the
feat/enrolment-script
branch
from
e1e31c1 to
72bbba9
Compare
|
Build succeeded. ✔️ pre-commit SUCCESS in 1m 03s |
|
regate |
|
Build succeeded (gate pipeline). ✔️ pre-commit SUCCESS in 1m 10s |
github-project-automation
bot
moved this from In review
to Done
in Packit pull requests
3 participants
Related to packit/deployment#679
Merge after packit/deployment#679