-
Notifications
You must be signed in to change notification settings - Fork 66
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
[release-4.10] WINC-818: [test] Fix upgradeable condition test #1339
[release-4.10] WINC-818: [test] Fix upgradeable condition test #1339
Conversation
@saifshaikh48: No Bugzilla bug is referenced in the title of this pull request. 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. |
Thanks for opening this @saifshaikh48 |
/cherry-pick release-4.9 The 4.9 cherry-pick needs #1340 to merge first (missing backport causing merge conflicts). |
@saifshaikh48: once the present PR merges, I will cherry-pick it on top of release-4.9 in a new PR and assign it to you. 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 commit fixes an issue where the upgradeable condition was not being tested in the e2e test suite. This was the case because the OPERATOR_CONDITION_NAME env var is not being set in the CI env when using the OLM upgrade workflow. The recommendation to fix this was to instead get it from the deployment spec as the condition is present once the operator is deployed. The cache client had to be replaced with the OLM client to get the operator condition using current context. (cherry picked from commit 0ffe33d) (cherry picked from commit cee94c5)
go get github.com/operator-framework/operator-lifecycle-manager/pkg/api/client/clientset/versioned go mod tidy -go=1.16 && go mod tidy -go=1.17 && go mod vendor
0a2544a
to
242eb13
Compare
/test lint |
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: saifshaikh48, sebsoto 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 |
@saifshaikh48: all tests passed! 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. |
@saifshaikh48: #1339 failed to apply on top of branch "release-4.9":
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 is a manual cherry-pick of #1318
Updated vendor commit for conflicts.