Skip to content
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

document maintenance of kubernetes-milestone-maintainers GH team #2424

Closed
tpepper opened this issue Jul 26, 2018 · 6 comments
Closed

document maintenance of kubernetes-milestone-maintainers GH team #2424

tpepper opened this issue Jul 26, 2018 · 6 comments
Assignees
Labels
sig/release Categorizes an issue or PR as relevant to SIG Release.

Comments

@tpepper
Copy link
Member

tpepper commented Jul 26, 2018

Who should be added to and removed from the kubernetes-milestone-maintainers GH team, and when, needs documented.

https://github.com/kubernetes/sig-release/blob/1b389936697e0881dfc343f634da2d37f03604ff/release-team/role-handbooks/ci-signal/README.md previously referred to but that and https://github.com/kubernetes/community/blob/master/project-managers/README.md#joining-the-group now 404's as #2357 removed it. The documentation there perhaps was more about joining the kubernetes-pm team, but should have been superseded by a process for kubernetes-milestone-maintainers.

It's now possible (see kubernetes/test-infra@19b225c) for repos to have their own milestone maintainers lists, but kubernetes-milestone-maintainers remains the default and is managed in an ad hoc by SIG-Release and the release team lead. But that's likely excessive in most cases.

Probably a simple process here works fine if it’s a limited number of people per sig/repo and we have a simple process like sig lead tells release lead who to add/remove…something light weight could work just fine I reckon. Only if sigs are rapidly adding/removing lots of people, then probably better for them to manage their own milestone team.

@k8s-ci-robot k8s-ci-robot added the needs-sig Indicates an issue or PR lacks a `sig/foo` label and requires one. label Jul 26, 2018
@tpepper
Copy link
Member Author

tpepper commented Jul 26, 2018

@tpepper
Copy link
Member Author

tpepper commented Jul 26, 2018

/sig pm
/sig release

@k8s-ci-robot k8s-ci-robot added sig/pm sig/release Categorizes an issue or PR as relevant to SIG Release. and removed needs-sig Indicates an issue or PR lacks a `sig/foo` label and requires one. labels Jul 26, 2018
@justaugustus
Copy link
Member

Great minds, @tpepper: kubernetes/enhancements#590 (comment)

@justaugustus
Copy link
Member

RFC for milestone-maintainers notes: kubernetes/sig-release#239

@justaugustus
Copy link
Member

/assign

@justaugustus
Copy link
Member

Added in kubernetes/sig-release#239
/close

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
sig/release Categorizes an issue or PR as relevant to SIG Release.
Projects
None yet
Development

No branches or pull requests

3 participants