Skip to content

WIP: extract sig data into file - #451

Draft
dhiller wants to merge 1 commit into
kubevirt:mainfrom
dhiller:experiment-per-dir-yaml
Draft

WIP: extract sig data into file#451
dhiller wants to merge 1 commit into
kubevirt:mainfrom
dhiller:experiment-per-dir-yaml

Conversation

@dhiller

@dhiller dhiller commented Jul 30, 2026

Copy link
Copy Markdown
Contributor

What this PR does / why we need it:

Extracts the sig data for sig-ci into a separate file. Also adds approvers and reviewers into the file - aiming towards making that file the single source of truth.

This could serve as a building block for OWNERS_ALIASES sync and also automated generation of GitHub teams.

Furthermore this makes the groups (SIGs, WGs, UGs, Committees) more autonomous, since SIG chairs can approve the changes themselves, once they have extracted the group.yaml into their group directory.

Note

Leaves the original file sigs.yaml for backwards compatibility, however the goal is to entirely remove that one and only use sigs.ReadFile()

Note

Currently we require to run make generate in order to have updates rendered into the sig-list.md, however that should be done as a postsubmit job instead.

Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Related to kubevirt/project-infra#5317

Extracts the sig data for sig-ci into a separate file. Also adds
approvers and reviewers into the file.

Aiming towards making that file the single source of truth.

Leaves the original file for backwards compatibility, however the goal
is to entirely remove that one and only use sigs.ReadFile()

This could serve as a building block for OWNERS_ALIASES sync and
also automated generation of GitHub teams.

Signed-off-by: Daniel Hiller <dhiller@redhat.com>
@kubevirt-prow

kubevirt-prow Bot commented Jul 30, 2026

Copy link
Copy Markdown

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

@kubevirt-prow kubevirt-prow Bot added do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. dco-signoff: yes Indicates the PR's author has DCO signed all their commits. labels Jul 30, 2026
@kubevirt-prow

kubevirt-prow Bot commented Jul 30, 2026

Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign vladikr for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@kubevirt-prow kubevirt-prow Bot added the size/L label Jul 30, 2026
@dhiller

dhiller commented Jul 30, 2026

Copy link
Copy Markdown
Contributor Author

@kubevirt-prow

kubevirt-prow Bot commented Aug 7, 2026

Copy link
Copy Markdown

PR needs rebase.

Details

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-sigs/prow repository.

@kubevirt-prow kubevirt-prow Bot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Aug 7, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dco-signoff: yes Indicates the PR's author has DCO signed all their commits. do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. size/L

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant