Add first-round drafts for k8s 1.12 Release Team #168
Conversation
Signed-off-by: Stephen Augustus <foo@agst.us>
|
@justaugustus: GitHub didn't allow me to request PR reviews from the following users: kacole2. Note that only kubernetes members and repo collaborators can review this PR, and authors cannot review their own PRs. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
k8s-ci-robot
added
cncf-cla: yes
Signed-off-by: Stephen Augustus <foo@agst.us>
|
/sig release |
k8s-ci-robot
added
the
sig/release
Signed-off-by: Stephen Augustus <foo@agst.us>
Signed-off-by: Stephen Augustus <foo@agst.us>
* Docs Lead: Zach Arnold * Docs Shadow: Samuel Tauil Signed-off-by: Stephen Augustus <foo@agst.us>
k8s-ci-robot
added
size/S
Signed-off-by: Stephen Augustus <foo@agst.us>
| | Release Notes ||| | ||
| | Communications ||| | ||
| | Communications | Kaitlyn Barnard ([@kbarnard10](https://github.com/kbarnard10)) || | ||
| | Patch Release Manager ||| |
There was a problem hiding this comment.
I'd like to be the Patch Release Manager
There was a problem hiding this comment.
Sounds good, @feiskyer. Thanks for volunteering!
|
Can we add "PR Triage" as a position? I nominate @dims for lead. I'll create a PR for the position. |
|
@jberkus +1 to |
Signed-off-by: Stephen Augustus <foo@agst.us>
Signed-off-by: Stephen Augustus <foo@agst.us>
|
If we're proposing adding more positions, @jberkus, I would suggest also adding a role focused on upgrade/downgrade testing |
Signed-off-by: Stephen Augustus <foo@agst.us>
|
/assign @tpepper |
|
Pending filling all lead roles... |
k8s-ci-robot
added
the
do-not-merge/hold
|
Regarding adding PR Triage: depending on how the future happens there's shifting overlaps between feature/PM oversight, issue triage, PR triage, and CI signal. MASSIVE CI signal attention seems to have lead to notably less issues in 1.11. The removal of requiring issues for PRs seems to have lead to less issues in 1.10 and 1.11 as well. But both of these reducing issue counts and issue connection to the milestone means less artifacts visible to the release team. In this context, while it incurs overhead on staffing, I like having these roles as separate. More pairs of eyeballs across this set makes for a more diverse risk management stance for the overall release. |
Signed-off-by: Stephen Augustus <foo@agst.us>
Signed-off-by: Stephen Augustus <foo@agst.us>
Signed-off-by: Stephen Augustus <foo@agst.us>
| | Communications ||| | ||
| | Patch Release Manager ||| | ||
| | Lead | Tim Pepper ([@tpepper](https://github.com/tpepper)) | Aishwarya Sundar ([@AishSundar](https://github.com/AishSundar)) | | ||
| | Features | Stephen Augustus ([@justaugustus](https://github.com/justaugustus)) | Kendrick Coleman ([@kacole2](https://github.com/kacole2)), Robert Sandoval ([@robertsandoval](https://github.com/robertsandoval)), Rajendar Talatam ([@rajendar38](https://github.com/rajendar38)) | |
There was a problem hiding this comment.
Is it fine that we'll have multiple shadows for this role?
Not against it, just would like to clarify.
There was a problem hiding this comment.
Yes, we're trying to overstaff this release cycle as:
- We're doing it much earlier than usual, so if prior commitments prevent people from participating, we will still be reasonably staffed.
- More eyes on every role should provide better coverage and allow us to iterate faster on improvements.
| | Lead | Tim Pepper ([@tpepper](https://github.com/tpepper)) | Aishwarya Sundar ([@AishSundar](https://github.com/AishSundar)) | | ||
| | Features | Stephen Augustus ([@justaugustus](https://github.com/justaugustus)) | Kendrick Coleman ([@kacole2](https://github.com/kacole2)), Robert Sandoval ([@robertsandoval](https://github.com/robertsandoval)), Rajendar Talatam ([@rajendar38](https://github.com/rajendar38)) | | ||
| | CI Signal || Christian Hernandez ([@christianh814](https://github.com/christianh814)) | | ||
| | Test Infra | Cole Wagner ([@cjwagner](https://github.com/cjwagner)) | Amit Watve ([@amwat](https://github.com/amwat)), Paul Renato ([@prwitt](https://github.com/prwitt)) | |
There was a problem hiding this comment.
The same question (Is it fine that we'll have multiple shadows for this role?)
Signed-off-by: Stephen Augustus <foo@agst.us>
Signed-off-by: Stephen Augustus <foo@agst.us>
Signed-off-by: Stephen Augustus <foo@agst.us>
Signed-off-by: Stephen Augustus <foo@agst.us>
Signed-off-by: Stephen Augustus <foo@agst.us>
|
I'm trying to track down a branch manager, but this may as well merge in the meantime for visibility. /lgtm |
k8s-ci-robot
added
the
lgtm
|
Also for the record, we discussed back and forth and are going to continue with "Bug Triage" which will cover both GitHub issue and PR type artifacts. |
|
/hold cancel |
k8s-ci-robot
removed
the
do-not-merge/hold
|
/assign @jberkus |
|
Let's see if I have approve powers: /approve |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: jberkus, justaugustus, tpepper 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 |
k8s-ci-robot
added
the
approved
ref: #167
/assign @tpepper @jberkus @jdumars
/cc @tpepper @AishSundar @kacole2
Signed-off-by: Stephen Augustus foo@agst.us