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

sig-observability: Update ownerships #11249

Conversation

machadovilaca
Copy link
Member

@machadovilaca machadovilaca commented Feb 14, 2024

What this PR does

Before this PR:

We had no assigned observability reviewer, nor approvers/reviewers and sig/observability labeling in observability packages

After this PR:

Owner files in:

  • pkg/monitoring
  • tests/monitoring
  • tests/libmonitoring
  • hack/prom-rule-ci

Fixes #

Why we need it and why it was done in this way

The following tradeoffs were made:

The following alternatives were considered:

Links to places where the discussion took place:

Special notes for your reviewer

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.

Release note

NONE

/cc @enp0s3 @dhiller @fabiand

@kubevirt-bot kubevirt-bot added release-note-none Denotes a PR that doesn't merit a release note. dco-signoff: yes Indicates the PR's author has DCO signed all their commits. size/XS area/monitoring labels Feb 14, 2024
Copy link
Contributor

@enp0s3 enp0s3 left a comment

Choose a reason for hiding this comment

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

@machadovilaca Hi, can you please add yourself to these as well:

/hack/prom-rule-ci
/tests/monitoring

@machadovilaca machadovilaca changed the title pkg/monitoring: Update ownership sig-observability: Update ownerships Feb 14, 2024
@machadovilaca
Copy link
Member Author

@machadovilaca Hi, can you please add yourself to these as well:

/hack/prom-rule-ci
/tests/monitoring

added

@fabiand
Copy link
Member

fabiand commented Feb 20, 2024

Thanks @machadovilaca

Please

  1. extend the PR to also set labels accoridng to sig like here https://github.com/kubevirt/kubevirt/pull/11225/files#diff-c393dd2b18a8a597d895940893e574d6895122c4bf3ce6e2a78dbd9ca0cdd66fR17-R18
  2. Updat ethe PR description to be meaningful

@machadovilaca machadovilaca force-pushed the add-sig-observability-to-pkg-monitoring branch from 7f623b2 to 08a70c3 Compare February 22, 2024 11:58
@machadovilaca
Copy link
Member Author

@fabiand updated

@iholder101
Copy link
Contributor

Thank you @machadovilaca!

I think it's essential to assign an approver to sig-obvservability as well.
Perhaps @sradco could be a good fit?

@kubevirt-bot kubevirt-bot added the sig/buildsystem Denotes an issue or PR that relates to changes in the build system. label Mar 21, 2024
@machadovilaca
Copy link
Member Author

@enp0s3 @fabiand can you take a new look?

OWNERS_ALIASES Outdated Show resolved Hide resolved
Signed-off-by: João Vilaça <jvilaca@redhat.com>
Signed-off-by: João Vilaça <jvilaca@redhat.com>
Signed-off-by: João Vilaça <jvilaca@redhat.com>
@machadovilaca machadovilaca force-pushed the add-sig-observability-to-pkg-monitoring branch from 01887ed to 63bb8f1 Compare March 21, 2024 13:05
@enp0s3
Copy link
Contributor

enp0s3 commented Mar 21, 2024

/approve

@kubevirt-bot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: enp0s3

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 /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@kubevirt-bot kubevirt-bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Mar 21, 2024
@iholder101
Copy link
Contributor

Looks good to me.
Holding to let @fabiand and others express their opinions and objections, if any.

/lgtm
/hold

@kubevirt-bot kubevirt-bot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Mar 21, 2024
@kubevirt-bot kubevirt-bot added the lgtm Indicates that a PR is ready to be merged. label Mar 21, 2024
@kubevirt-commenter-bot
Copy link

Required labels detected, running phase 2 presubmits:
/test pull-kubevirt-e2e-windows2016
/test pull-kubevirt-e2e-kind-1.27-vgpu
/test pull-kubevirt-e2e-kind-sriov
/test pull-kubevirt-e2e-k8s-1.29-ipv6-sig-network
/test pull-kubevirt-e2e-k8s-1.27-sig-network
/test pull-kubevirt-e2e-k8s-1.27-sig-storage
/test pull-kubevirt-e2e-k8s-1.27-sig-compute
/test pull-kubevirt-e2e-k8s-1.27-sig-operator
/test pull-kubevirt-e2e-k8s-1.28-sig-network
/test pull-kubevirt-e2e-k8s-1.28-sig-storage
/test pull-kubevirt-e2e-k8s-1.28-sig-compute
/test pull-kubevirt-e2e-k8s-1.28-sig-operator

@machadovilaca
Copy link
Member Author

ping @fabiand

@fabiand
Copy link
Member

fabiand commented Apr 3, 2024

Thanks @machadovilaca please link to a SIG Charter proposal as other SIGs have done in this thread: https://groups.google.com/g/kubevirt-dev/c/9hOLqb9W2-Y

@fabiand
Copy link
Member

fabiand commented Apr 3, 2024

I'm

/hold

ing until then, but generally

/lgtm

@machadovilaca
Copy link
Member Author

sig-observability charter:
mailing list - https://groups.google.com/g/kubevirt-dev/c/9hOLqb9W2-Y
PR - kubevirt/community#282

@fabiand
Copy link
Member

fabiand commented Apr 3, 2024

/unhold

as a charter got posted

@kubevirt-bot kubevirt-bot removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Apr 3, 2024
@kubevirt-bot kubevirt-bot merged commit 48f0f07 into kubevirt:main Apr 3, 2024
40 checks passed
@avlitman
Copy link
Contributor

/cherry-pick release-1.2

@kubevirt-bot
Copy link
Contributor

@avlitman: new pull request created: #11705

In response to this:

/cherry-pick release-1.2

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/test-infra repository.

@avlitman
Copy link
Contributor

/cherry-pick release-1.1

@avlitman
Copy link
Contributor

/cherry-pick release-1.0

@kubevirt-bot
Copy link
Contributor

@avlitman: new pull request created: #11706

In response to this:

/cherry-pick release-1.1

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/test-infra repository.

@kubevirt-bot
Copy link
Contributor

@avlitman: new pull request created: #11707

In response to this:

/cherry-pick release-1.0

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/test-infra repository.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. area/monitoring dco-signoff: yes Indicates the PR's author has DCO signed all their commits. lgtm Indicates that a PR is ready to be merged. release-note-none Denotes a PR that doesn't merit a release note. sig/buildsystem Denotes an issue or PR that relates to changes in the build system. size/M
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants