Task #3379
closedEnable required checks on Github for core and pulp_file
0%
Description
Make the following checks required:
- PR review
- docs builders check
- Travis CI passing
- "default" which is the Jenkins test runner check
For the following repos and branches:
- 'pulp' repo 'master' branch
- 'pulp' repo '3.0-dev' branch
- 'pulp_file' repo 'master' branch
In terms of a disable/reenable process:
From the github docs it looks like an 'Administrator' of a repository can enable/disable the required checks. Currently the following people have that permissions: @ipanova, rchan, @jortel, ttereshc, and bmbouter
To temporarily disable:
1. Contact one of ^ admins via pulp-dev and irc that a check is failing permanently
2. Discuss what the issue is and make a decision to disable the checks or not.
3. If disabling, disable via github and make and issue to re-enable
4. notify pulp-dev with an update of the changes and the issue link
5. work to re-enable checks as soon as possible
We should put ^ and an overall update of the latest required checks into the pulp2 and pulp3 docs. We can put those into these locations:
pulp2: https://docs.pulpproject.org/dev-guide/contributing/merging.html#pull-requests
pulp3: https://docs.pulpproject.org/dev-guide/contributing/merging.html#pull-requests
Updated by bmbouter about 6 years ago
- Description updated (diff)
rewriting to include the 'default' check and providing some better organizing.
Updated by rchan about 6 years ago
I don't see any mention of the "ability to temporarily disable them" mentioned in the pulp-dev discussion, will this be handled via a different process (can we specify) or just inadvertently left out?
Updated by bmbouter about 6 years ago
That is an oversight on my part from haste. We talked about that on the mailing list and I need to add it here. I'll update this ticket with a plan for comment.
Updated by bmbouter about 6 years ago
Some ideas on where to put these docs:
For pulp3 I think it's here: https://github.com/pulp/pulp/blob/3.0-dev/docs/contributing/git.rst
For pulp2 here: https://github.com/pulp/pulp/blob/master/docs/dev-guide/contributing/merging.rst
Updated by bmbouter about 6 years ago
- Description updated (diff)
Adding info about the temporary disabling process
Updated by bmbouter about 6 years ago
- Sprint Candidate changed from No to Yes
marking as a sprint candidate in hopes that it gets groomed
Updated by bmbouter about 6 years ago
- Subject changed from Enable strict checks on Github for core and pulp_file to Enable required checks on Github for core and pulp_file
Updated by bmbouter almost 6 years ago
- Status changed from NEW to CLOSED - WONTFIX
- Sprint Candidate set to No
With the recent enabling of continuous integration at merge time this ticket is "aged out". The current state of Pulp continuous integration work is described here: https://pulp.plan.io/projects/pulp/wiki/Continuous_Delivery_of_Pulp_3/
With CI as an already enabled required check as part of a totally separate effort, it happened to address the original need for Pulp3. I suggest leaving Pulp2 alone in this area and just closing this ticket.
Please reopen if anyone feels differently.