community
community copied to clipboard
kubevirt
community: generate sig-list.md
What this PR does / why we need it:
Creates an overview over all the user groups, sigs and committees that are defined via sigs.yaml. Output is part of this PR as sig-list.md.
Also corrects a couple of labels for the sigs.
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
/cc @aburdenthehand @xpivarc
Checklist
This checklist is not enforcing, but it's a reminder of items that could be relevant to every PR. Approvers are expected to review this list.
- [ ] Design: A design document was considered and is present (link) or not required
- [ ] PR: The PR description is expressive enough and will help future contributors
- [ ] Code: Write code that humans can understand and Keep it simple
- [ ] Refactor: You have left the code cleaner than you found it (Boy Scout Rule)
- [ ] Upgrade: Impact of this change on upgrade flows was considered and addressed if required
- [ ] Testing: New code requires new unit tests. New features and bug fixes require at least on e2e test
- [ ] Documentation: A user-guide update was considered and is present (link) or not required. You want a user-guide update if it's a user facing feature / API change.
- [ ] Community: Announcement to kubevirt-dev was considered
Release note:
Cool! Great stuff!
Is there a way to ensure this list is not getting outdated over time?
Cool! Great stuff!
Is there a way to ensure this list is not getting outdated over time?
I thought of this as the first step - in another step we can set up a job that automatically updates via a PR whenever sigs.yaml gets changed.
As an alternative we could create a presubmit that tells people they need to call make generate as we do in k/kubevirt.
@aburdenthehand updated, thanks for the valuable input!
@xpivarc would you be able to give this another review?
This PR is waiting for three weeks now.
Hey @cwilkers @davidvossel @fabiand @rmohr @vladikr :wave:
Since @aburdenthehand is out for a while, is one of you able to take a quick look and approve this?
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: dhiller, vladikr
The full list of commands accepted by this bot can be found here.
The pull request process is described here
- ~~OWNERS~~ [vladikr]
Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment
