Revert "Set default messages & reconcile clusteroperator status conditions (#584) #586
Conversation
…tions (openshift#584)" This reverts commit 11e44f0. After this merged, some clusters are failing to install with this message: ``` unable to set initial cluster status: ClusterOperator.config.openshift.io "insights" is invalid: status.conditions.lastTransitionTime: Required value ``` OpenShift policy is to revert payload breaking changes immediately, and the team can build their fix on top of the unrevert.
|
@stbenjam Thanks. Yes I just noticed some problems as well. Thank you. |
|
/approve |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: stbenjam, tremes 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 |
openshift-ci
bot
added
the
approved
|
@tremes Thank you for understanding. To reintroduce this, you probably want to get a good run against e2e-metal-ipi-ovn-ipv6: openshift/release#26721 adds it to this repo |
|
/retest-required Please review the full test history for this PR and help us cut down flakes. |
|
opened after the latest merge to master. upgrade failure was unrelated. merging to fix master. |
|
@stbenjam: 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. |
This reverts commit 11e44f0. After this
merged, some clusters are failing to install with this message:
OpenShift policy is to revert payload breaking changes immediately, and
the team can build their fix on top of the unrevert.