github-management: add docs for master to main migration #5609
Conversation
k8s-ci-robot
added
the
do-not-merge/hold
k8s-ci-robot
added
cncf-cla: yes
k8s-ci-robot
added
area/github-management
nikhita
force-pushed
the
main-migration-docs-01
branch
from
fd3deb8 to
de33b48
Compare
|
I'll review tomorrow, but I'm also fine if this merges before then; I can open a followup PR if I have specific concerns. |
@spiffxp I'm good with waiting until you have a chance to review :) |
| Check if any of the prowjobs (presubmits, postsubmits or periodics) | ||
| explicitly mention the `master` branch. | ||
|
|
||
| - [ ] If a prowjob triggers on the `master` branch (`branches` field |
There was a problem hiding this comment.
I'm fine with this as written but still think / hope someone could do this en-masse and save folks the trouble
| of the prowjob), add the `main` branch to the list | ||
| (see [kubernetes/test-infra#20665] for an example). | ||
|
|
||
| - [ ] If a prowjob mentions the `master` branch in `base_ref`, |
There was a problem hiding this comment.
Same here. I hope to eliminate the necessity of this step with updates to pod utils and bootstrap
There was a problem hiding this comment.
I hope to eliminate the necessity of this step with updates to pod utils and bootstrap
just for the record, issues tracking updates to:
| PR status contexts to the new job name but this has not been tested yet. | ||
| This _may_ also impact testgrid history post-migration. | ||
|
|
||
| - [ ] If a prowjob calls scripts or code in your repo that explicitly |
There was a problem hiding this comment.
I would prefer seeding this with a few recipes/snippets on how to update scripts to refer to the default branch instead of hard coding and requiring a change in lockstep.
I'll offer some when at a keyboard
There was a problem hiding this comment.
I'll offer some when at a keyboard
I'm going to leave this to you :)
There was a problem hiding this comment.
Since I didn't get these in before your latest round of updates, I'm going to do this as followup
nikhita
force-pushed
the
main-migration-docs-01
branch
from
de33b48 to
521469b
Compare
This document is meant to be used by repos without too much traffic, if they wish to migrate the default branch from master to main.
nikhita
force-pushed
the
main-migration-docs-01
branch
from
521469b to
198374d
Compare
k8s-ci-robot
added
the
lgtm
|
/hold cancel |
k8s-ci-robot
removed
the
do-not-merge/hold
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: nikhita, spiffxp The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Ref: kubernetes/org#2222
This document is meant to be used by repos without too much traffic,
if they wish to migrate the default branch from master to main.
/hold
for review
/assign @spiffxp @mrbobbytables @cblecker