-
Notifications
You must be signed in to change notification settings - Fork 39
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Harmonise release leads table & roster #260
Conversation
Questions about the roster:
|
I see the motivation here. However, as far as I remember we (client) were always carried by the appointed release leads from either Serving or Eventing WGs. Hence there are no more special release steps for Client, plugins or Functions repositories. I wonder if we really need the split to WG groups per historical context. Or we should merge pools of available release leads (I assume usually WG leads apply here) and pick the rotation from a single one. I would like to discuss it with TOC as well. If there are any opinions. /cc @knative/technical-oversight-committee |
I think that is a very good point and I like your proposal. |
If I remember right, part of the rationale for having someone from Serving and Eventing on the release was in case there were Serving- or Eventing-specific issues that popped up, for example, something in the autoscaler that would be difficult for someone not familiar with Serving to fix. Not sure how often those kinds of issues are still occurring (if at all), but assuming someone from the WG could be available at some point to assist if/when those types of things pop up, then I could see not needing them to be a formal release lead... |
There's an issue here discussion this point: #32 tl;dr we just need a roster of folks to help shepherd the release. Any issues with particular repos should then require the release leads to pull in the respective WG leads to help. eg. test is flaking etc. |
So assuming, that in the future we'll only have one or two release leads shepherding the release and folks from the respective WGs joining to take care of their repos. I updated the PR to reflect:
WDYT? |
I'm willing to stay on a rotation; historically (when this was started) there was a bit of unsteadiness in the releases that I think has been ironed out since. |
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: dprotaso, ReToCode 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 |
Changes
I think we should have client & functions also on this table, as you folks are always also a part of the release.
/assign @dsimansk