Skip to content

Conversation

@dhiller
Copy link
Contributor

@dhiller dhiller commented Jan 28, 2026

What this PR does / why we need it:

Remove @brianmcarey from chairs of sig-ci since he has left the project 🥹

Also update the meeting times for sig-ci meeting and quarantine catchup meeting.

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

Also update the meeting times for sig-ci meeting and quarantine catchup
meeting.

Signed-off-by: Daniel Hiller <dhiller@redhat.com>
@kubevirt-bot kubevirt-bot added the dco-signoff: yes Indicates the PR's author has DCO signed all their commits. label Jan 28, 2026
@kubevirt-bot
Copy link
Contributor

[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 xpivarc 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

Copy link
Contributor

@iholder101 iholder101 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm

Comment on lines +324 to +331
time: "10:00"
tz: CET
frequency: weekly
url: https://zoom.us/j/96583958896
archive_url: https://docs.google.com/document/d/1pWg6JvjStffoZ0NnVk4ALQOs3MYOdHtBpkLBlJlHTqA/edit
- description: KubeVirt tests quarantine catch-up
day: Wednesday
time: "08:45"
time: "09:45"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Are these related? Just making sure it's not a typo

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actually this is just a fix, since the meetings always happens on 9:45 AM CE(S)T. This is correctly seen in the KubeVirt calendar but wrong here.

@kubevirt-bot kubevirt-bot added lgtm Indicates that a PR is ready to be merged. and removed lgtm Indicates that a PR is ready to be merged. labels Jan 28, 2026
@kubevirt-bot
Copy link
Contributor

New changes are detected. LGTM label has been removed.

@kubevirt-bot
Copy link
Contributor

There are empty aliases in OWNER_ALIASES, cleanup is advised.

sig-ci-reviewers:
- brianmcarey
- dhiller
- dollierp
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

FYI @dollierp

@dhiller
Copy link
Contributor Author

dhiller commented Jan 30, 2026

/cc @brianmcarey

@kubevirt-bot
Copy link
Contributor

There are empty aliases in OWNER_ALIASES, cleanup is advised.

Adds a section into the sig-list.md that mentions the emeritus leads,
should they be present.

This change specifically moves @brianmcarey into the emeritus_approvers
section in the OWNERS and removes him from the approvers sections in
OWNERS_ALIASES. Also an emeritus_leads section is added for him.

Signed-off-by: Daniel Hiller <dhiller@redhat.com>
Copy link
Member

@brianmcarey brianmcarey left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

thanks @dhiller

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. size/M

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants