-
Notifications
You must be signed in to change notification settings - Fork 605
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
Wait for cluster version resource to load #3091
Wait for cluster version resource to load #3091
Conversation
@@ -167,6 +167,7 @@ const ClusterAlerts = withDashboardResources( | |||
const alerts = getAlerts(alertsResponse); | |||
|
|||
const cv = _.get(resources.cv, 'data') as ClusterVersionKind; | |||
const cvLoaded = _.get(resources.cv, 'loaded'); |
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.
We probably want to check the CLUSTER_VERSION
flag before watching this resource.
@rawagner It looks like the dashboards page is available on native k8s. I suspect a lot of things will fail when not running on OpenShift. |
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.
/lgtm
Let's get this fix in, but we might look at adding flag checks for resources before we request them in a follow on
/retest |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: rawagner, spadgett 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 |
known olm flake /retest |
/retest |
/retest Please review the full test history for this PR and help us cut down flakes. |
@spadgett this will fix the case when alerts are loaded (and there arent any) and the status card will show
No cluster alerts or messages
and after a while Cluster Version resource is loaded and the card will show available update.