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
OCPVE-632: add capability annotations to manifests #856
OCPVE-632: add capability annotations to manifests #856
Conversation
Thanks for this @qJkee. Could you look at the test failures? |
fa42e0f
to
b733d35
Compare
@flavianmissi |
thank you @qJkee !
Gotcha. Is the plan to change installer and CVO before merging this PR? |
Also - do we need resources created at runtime by the operator to also have this capability? I'm assuming we don't need that but want to confirm. |
@flavianmissi No, just the install manifests because they disable the operator completely |
should we put this PR on hold then to signalize that we are waiting? |
/hold until we merge cap name to CVO |
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.
Does the manifests/01-registry-credentials-request-powervs.yaml
file also need to get the capabilities annotation?
Looks good otherwise
I think yes, because in |
/retest |
1 similar comment
/retest |
Ci is not doing well |
/jira-refresh |
/label qe-approved |
b733d35
to
107e74e
Compare
/unhold |
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: flavianmissi, qJkee 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 |
/label docs-approved |
/test e2e-aws-operator |
/label px-approved |
/hold because of #899 |
/retest |
@qJkee: This pull request references Jira Issue OCPBUGS-17806, which is valid. 3 validation(s) were run on this bug
The bug has been updated to refer to the pull request using the external bug tracker. 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. |
@qJkee: This pull request references OCPVE-632 which is a valid jira issue. 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. |
/label acknowledge-critical-fixes-only |
/unhold |
/test e2e-hypershift |
/retest |
e2e-aws-operator run failed on a build-cluster hiccup. /retest-required |
same failure as previous |
/retest-required |
test failure seems unrelated |
@qJkee: The following tests failed, say
Full PR test history. Your PR dashboard. 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. I understand the commands that are listed here. |
4d68ce9
into
openshift:master
Add capability annotation to manifests to make image registry optional