-
Notifications
You must be signed in to change notification settings - Fork 153
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
Temporary workaround to trigger upgrade #1323
Conversation
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
Pull Request Test Coverage Report for Build 1303655222
💛 - Coveralls |
/retest |
/override-bot |
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.
/approve
hco-e2e-image-index-aws lane succeeded. |
@hco-bot: Overrode contexts on behalf of hco-bot: ci/prow/hco-e2e-image-index-gcp 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. |
hco-e2e-image-index-aws lane succeeded. |
@hco-bot: Overrode contexts on behalf of hco-bot: ci/prow/hco-e2e-image-index-azure, ci/prow/okd-hco-e2e-upgrade-index-aws 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. |
hack/upgrade-test-index-image.sh
Outdated
@@ -150,6 +150,10 @@ HCO_SUBSCRIPTION=$(${CMD} get subscription -n ${HCO_NAMESPACE} -o name) | |||
OLD_INSTALL_PLAN=$(oc -n "${HCO_NAMESPACE}" get "${HCO_SUBSCRIPTION}" -o jsonpath='{.status.installplan.name}') | |||
${CMD} patch ${HCO_SUBSCRIPTION} -n ${HCO_NAMESPACE} -p "{\"spec\": {\"channel\": \"${TARGET_CHANNEL}\"}}" --type merge | |||
|
|||
# TEMP HACK TO UNLOCK UPGRADE | |||
sleep 15 | |||
${CMD} delete -n openshift-operator-lifecycle-manager "$(oc get pod -n openshift-operator-lifecycle-manager -l app=olm-operator -o name)" |
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.
${CMD} delete -n openshift-operator-lifecycle-manager "$(oc get pod -n openshift-operator-lifecycle-manager -l app=olm-operator -o name)" | |
${CMD} delete -n openshift-operator-lifecycle-manager "$(${CMD} get pod -n openshift-operator-lifecycle-manager -l app=olm-operator -o name)" |
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.
thanks.
used for loop instead to avoid duplicated lines
hack/upgrade-test-index-image.sh
Outdated
# TEMP HACK TO UNLOCK UPGRADE | ||
sleep 15 | ||
${CMD} delete -n openshift-operator-lifecycle-manager "$(oc get pod -n openshift-operator-lifecycle-manager -l app=olm-operator -o name)" | ||
${CMD} delete -n openshift-operator-lifecycle-manager "$(oc get pod -n openshift-operator-lifecycle-manager -l app=catalog-operator -o name)" |
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.
${CMD} delete -n openshift-operator-lifecycle-manager "$(oc get pod -n openshift-operator-lifecycle-manager -l app=catalog-operator -o name)" | |
${CMD} delete -n openshift-operator-lifecycle-manager "$(${CMD} get pod -n openshift-operator-lifecycle-manager -l app=catalog-operator -o name)" |
…upgrade Signed-off-by: orenc1 <ocohen@redhat.com>
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
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.
/approve
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: nunnatsa 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 |
hco-e2e-image-index-aws lane succeeded. |
@hco-bot: Overrode contexts on behalf of hco-bot: ci/prow/hco-e2e-image-index-azure, ci/prow/hco-e2e-image-index-gcp, ci/prow/okd-hco-e2e-upgrade-index-gcp 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. |
@orenc1: 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. |
/override-bot |
hco-e2e-kv-smoke-azure lane succeeded. |
1 similar comment
hco-e2e-kv-smoke-azure lane succeeded. |
/override ci/prow/hco-e2e-kv-smoke-gcp |
checking if openshift-ci bot is alive /shrug |
@hco-bot: hco-bot unauthorized: /override is restricted to Repo administrators, approvers in top level OWNERS file. 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. |
@hco-bot: hco-bot unauthorized: /override is restricted to Repo administrators, approvers in top level OWNERS file. 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. |
@orenc1: Overrode contexts on behalf of orenc1: ci/prow/hco-e2e-kv-smoke-gcp 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. |
/cherry-pick release-1.5 |
@tiraboschi: new pull request created: #1552 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. |
Revert an hack on upgrade lanes introduced in kubevirt#1323 to bypass a possible upgrade bug on the OLM side. Signed-off-by: Simone Tiraboschi <stirabos@redhat.com>
Revert an hack on upgrade lanes introduced in #1323 to bypass a possible upgrade bug on the OLM side. Signed-off-by: Simone Tiraboschi <stirabos@redhat.com>
Restart OLM pods (
olm-operator
andcatalog-operator
) just after patching the subscription to the new channel contains v100.0.0 to trigger OLM upgrade process.Temporary workaround until root cause will be identified and fixed.
Release note: