Skip to content
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

Created new extension to show health status from K8s Resources #4840

Conversation

afreen23
Copy link
Contributor

@afreen23 afreen23 commented Mar 27, 2020

The status card health subsystem shows status fetched from prometheus and url.
This commit extends that to add an extension to show status from any K8s resources.
The new extension is Dashboards/Overview/Health/Resource

Followup PR to consume the extension #4824

cc @rawagner

@openshift-ci-robot openshift-ci-robot added component/core Related to console core functionality component/dashboard Related to dashboard component/sdk Related to console-plugin-sdk labels Mar 27, 2020
@afreen23 afreen23 changed the title Created new extension to show health status from CRD Created new extension to show health status from K8s Resources Mar 30, 2020
The status card health subsystem shows status fetched from promethes and url.
This commit extends that to add an extension to show status from Resources.
The new extension is `Dashboards/Overview/Health/Resource`

Signed-off-by: Afreen Rahman <afrahman@redhat.com>
@rawagner
Copy link
Contributor

rawagner commented Apr 1, 2020

/lgtm

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Apr 1, 2020
@openshift-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: afreen23, rawagner

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 /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-ci-robot openshift-ci-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Apr 1, 2020
@openshift-bot
Copy link
Contributor

/retest

Please review the full test history for this PR and help us cut down flakes.

@openshift-merge-robot openshift-merge-robot merged commit 18c7703 into openshift:master Apr 1, 2020
@spadgett spadgett added this to the v4.5 milestone Apr 2, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. component/core Related to console core functionality component/dashboard Related to dashboard component/sdk Related to console-plugin-sdk lgtm Indicates that a PR is ready to be merged.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants