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
Bug 1872309: Rename "cluster_monitoring_operator" -> "cluster-monitoring-operator" #910
Bug 1872309: Rename "cluster_monitoring_operator" -> "cluster-monitoring-operator" #910
Conversation
96583b1
to
2f2559e
Compare
And I've filed openshift/cluster-version-operator#439 to remove the oversimplified regexp from the CVO's docs. |
2f2559e
to
886d998
Compare
The outgoing, underscore form landed in 9fd35c2 (#388, new in 4.2), so we could backport this to any of our not-yet-end-of-life release streams if folks wanted to. Personally, I doubt it needs backporting at all, since the only risk of the old naming is that a separate operator made the same mistake and set |
this generally lgtm, but i assume we need a BZ for this as we are in QA phase?! |
Does this impact down/upgrades in any way? I don't believe so but just want to make sure we are not running in any edge case. |
I asked about that a few weeks ago, but nobody has landed a bug requirement for master yet (see Tide saying
There's no relationship between manifest filenames between releases, so this rename has about the same pact there as the earlier #388. The only impacts I can see now are the unlikely overlap-fix from my previous comment. And I haven't audited a recent release, but I would be very surprised if we had an overlap like that today. This is just cosmetic, and a guard against low-risk, future overlap. |
/lgtm |
@wking according to tide this needs now a bugzilla 🤷♂️ |
/retitle Bug 1872309: Rename "cluster_monitoring_operator" -> "cluster-monitoring-operator" |
@wking: This pull request references Bugzilla bug 1872309, which is valid. The bug has been moved to the POST state. The bug has been updated to refer to the pull request using the external bug tracker. 3 validation(s) were run on this bug
In response to this:
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. |
/bugzilla refresh |
@simonpasquier: This pull request references Bugzilla bug 1872309, which is valid. 3 validation(s) were run on this bug
In response to this:
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. |
/retest Please review the full test history for this PR and help us cut down flakes. |
6 similar comments
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
3 similar comments
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
hold GitHub claims a conflict. |
/retest Please review the full test history for this PR and help us cut down flakes. |
10 similar comments
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
…g-operator" The regular expression for the component portion is [1]: [a-z0-9\-]+ or [2]: [a-zA-Z0-9]+(-[a-zA-Z0-9]+)*?) Either way, it cannot include underscores, so manifests like 0000_50_cluster_monitoring_operator_01-namespace.yaml end up in the 'cluster' component. With this change, they will end up in the 'cluster-monitoring-operator' component. The outgoing, underscore form landed in 9fd35c2 (manifests: prefix manifests used by CVO with a 0000_xx_NAME prefix, 2019-06-25, openshift#388). Generated with: $ rename cluster_monitoring_operator cluster-monitoring-operator manifests/*.yaml $ sed -i 's/cluster_monitoring_operator/cluster-monitoring-operator/g' Documentation/data-collection.md Makefile README.md hack/build-jsonnet.sh hack/build-jsonnet.sh hack/local-cmo.sh hack/telemeter_query.go [1]: https://github.com/openshift/cluster-version-operator/blob/71aef74480d199fe96a590f2f1e4e8056a9cb687/docs/dev/operators.md#what-is-the-order-that-resources-get-createdupdated-in [2]: https://github.com/openshift/cluster-version-operator/blob/71aef74480d199fe96a590f2f1e4e8056a9cb687/pkg/payload/task_graph.go#L21
886d998
to
f11007b
Compare
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: simonpasquier, wking 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 |
/retest Please review the full test history for this PR and help us cut down flakes. |
3 similar comments
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
@wking: All pull requests linked via external trackers have merged: Bugzilla bug 1872309 has been moved to the MODIFIED state. In response to this:
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. |
The regular expression for the component portion is
[a-z0-9\-]+
or[a-zA-Z0-9]+(-[a-zA-Z0-9]+)*?)
. Either way, it cannot include underscores, so manifests like0000_50_cluster_monitoring_operator_01-namespace.yaml
end up in thecluster
component. With this change, they will end up in thecluster-monitoring-operator
component.Generated with: