-
Notifications
You must be signed in to change notification settings - Fork 363
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 1915473: Annotate manifests for single-node-developer cluster profile #1003
Bug 1915473: Annotate manifests for single-node-developer cluster profile #1003
Conversation
@cfergeau hey, can you link to the enhancement? https://github.com/openshift/enhancements#482 as this link is a dead end. Thanks! |
7b0285c
to
2124dfc
Compare
Oh sorry for that, fixed now! It was referring to ttps://github.com/openshift/enhancements/pull/482 which has now been pushed https://github.com/openshift/enhancements/blob/master/enhancements/single-node-developer-cluster-profile.md |
2124dfc
to
942b184
Compare
942b184
to
83f920b
Compare
/retest |
1 similar comment
/retest |
@@ -5,6 +5,7 @@ metadata: | |||
controller-gen.kubebuilder.io/version: v0.2.4 | |||
include.release.openshift.io/ibm-cloud-managed: "true" | |||
include.release.openshift.io/self-managed-high-availability: "true" | |||
include.release.openshift.io/single-node-developer: "true" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
there's something not right in our tooling because this file (and the other CRDs) are generated from Jsonnet. The new annotation should be added to jsonnet/add-release-annotation.libsonnet
. I'm investigating why the CI didn't fail.
/retest |
This partially implements phase 1 of https://github.com/openshift/enhancements#482 and does not change behavior. Initially, all cluster-monitoring-operator manifests are included in the single-node-developer cluster profile. Follow-on PRs may exclude any of these that are not needed in the profile.
83f920b
to
a036fbe
Compare
/unhold |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: cfergeau, simonpasquier 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 |
@cfergeau: This pull request references Bugzilla bug 1915473, which is invalid:
Comment 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 |
@cfergeau: This pull request references Bugzilla bug 1915473, which is valid. The bug has been moved to the POST state. 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. |
8 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. |
@cfergeau: Some pull requests linked via external trackers have merged: The following pull requests linked via external trackers have not merged: These pull request must merge or be unlinked from the Bugzilla bug in order for it to move to the next state. Once unlinked, request a bug refresh with Bugzilla bug 1915473 has not 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. |
This partially implements phase 1 of
https://github.com/openshift/enhancements#482 and does not change behavior.
Initially, all cluster-monitoring-operator manifests are included in the
single-node-developer cluster profile. Follow-on PRs may exclude any of
these that are not needed in the profile.