Skip to content

Remove incorrect OpenShift version from cluster overview page#1065

Merged
openshift-merge-robot merged 1 commit intoopenshift:masterfrom
spadgett:remove-openshift-version
Jan 12, 2019
Merged

Remove incorrect OpenShift version from cluster overview page#1065
openshift-merge-robot merged 1 commit intoopenshift:masterfrom
spadgett:remove-openshift-version

Conversation

@spadgett
Copy link
Copy Markdown
Member

The OpenShift version endpoint no longer returns a version. The version is available on the cluster settings page where we get the ClusterVersion resource. Remove OpenShift version from the cluster overview page for now to avoid showing an incorrect value. We might want to add the correct version and cluster id to this page later.

When #1030 merges, we can connect to redux to get the cluster id (and possibly add version to redux as well).

/assign @rhamilto

We can later add version from the ClusterVersion resource.
@openshift-ci-robot openshift-ci-robot added size/M Denotes a PR that changes 30-99 lines, ignoring generated files. lgtm Indicates that a PR is ready to be merged. labels Jan 11, 2019
Copy link
Copy Markdown
Member

@rhamilto rhamilto left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm

@spadgett
Copy link
Copy Markdown
Member Author

jenkins rebuild

8 similar comments
@spadgett
Copy link
Copy Markdown
Member Author

jenkins rebuild

@spadgett
Copy link
Copy Markdown
Member Author

jenkins rebuild

@spadgett
Copy link
Copy Markdown
Member Author

jenkins rebuild

@spadgett
Copy link
Copy Markdown
Member Author

jenkins rebuild

@spadgett
Copy link
Copy Markdown
Member Author

jenkins rebuild

@spadgett
Copy link
Copy Markdown
Member Author

jenkins rebuild

@spadgett
Copy link
Copy Markdown
Member Author

jenkins rebuild

@spadgett
Copy link
Copy Markdown
Member Author

jenkins rebuild

@openshift-merge-robot openshift-merge-robot merged commit 67a6792 into openshift:master Jan 12, 2019
@spadgett spadgett deleted the remove-openshift-version branch January 13, 2019 22:07
@vojtechszocs vojtechszocs mentioned this pull request Jan 28, 2019
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm Indicates that a PR is ready to be merged. size/M Denotes a PR that changes 30-99 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants