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

KUBESAW-187: Have a unique label for our operators #597

Merged
merged 2 commits into from
Sep 19, 2024

Conversation

fbm3307
Copy link
Contributor

@fbm3307 fbm3307 commented Sep 19, 2024

Have a unique label instead of default controller-manger, so that it becomes easier to perform operations to only our operators
this is a predecessor for PR
KSCTL - kubesaw/ksctl#79

Similar PR
Host-Operator - codeready-toolchain/host-operator#1087

control-plane: controller-manager
control-plane: kubesaw-controller-manager
Copy link
Contributor

Choose a reason for hiding this comment

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

TBH, I'm not sure if this labels is not used somewhere else (eg in sanbox-sre). Let's keep the label there and add a new one instead

Copy link
Contributor Author

Choose a reason for hiding this comment

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

yes you are right, added new ones now.

Copy link

sonarcloud bot commented Sep 19, 2024

@fbm3307 fbm3307 changed the title KUBESAW-170: Have a unique label for our operators KUBESAW-187: Have a unique label for our operators Sep 19, 2024
Copy link
Contributor

@MatousJobanek MatousJobanek left a comment

Choose a reason for hiding this comment

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

/lgtm

Copy link

openshift-ci bot commented Sep 19, 2024

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: fbm3307, MatousJobanek

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

Copy link

codecov bot commented Sep 19, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 83.56%. Comparing base (53187a5) to head (f1376e6).
Report is 1 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #597   +/-   ##
=======================================
  Coverage   83.56%   83.56%           
=======================================
  Files          28       28           
  Lines        2604     2604           
=======================================
  Hits         2176     2176           
  Misses        288      288           
  Partials      140      140           

@openshift-merge-bot openshift-merge-bot bot merged commit f80135a into codeready-toolchain:master Sep 19, 2024
11 checks passed
MatousJobanek added a commit that referenced this pull request Sep 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants