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

Bug 1666231 - Update Cluster Settings page for ClusterVersion API changes #1100

Merged
merged 1 commit into from
Feb 5, 2019

Conversation

spadgett
Copy link
Member

@spadgett spadgett commented Jan 17, 2019

Change "Current Version" label to "Desired Version," which is more
accurate. Add an "Update History" table so that it's possible to see the
last completed update version.

https://bugzilla.redhat.com/show_bug.cgi?id=1666231

/hold

cc @smarterclayton @openshift/team-ux-review
/assign @TheRealJon

localhost_9000_settings_cluster 1

@openshift-ci-robot openshift-ci-robot added do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. size/L Denotes a PR that changes 100-499 lines, ignoring generated files. labels Jan 17, 2019
Copy link
Contributor

@serenamarie125 serenamarie125 left a comment

Choose a reason for hiding this comment

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

@spadgett should STATE column be STATUS? And should we have icons with the associated status values?

@spadgett
Copy link
Member Author

@spadgett should STATE column be STATUS? And should we have icons with the associated status values?

Good question. The values are "Completed" or "Partial," which is different than what we show for "Update Status" above. The API field is state, so I used "State" here. Let me know what you think.

@spadgett
Copy link
Member Author

jenkins rebuild

1 similar comment
@spadgett
Copy link
Member Author

jenkins rebuild

@openshift-ci-robot openshift-ci-robot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Feb 1, 2019
@openshift-ci-robot openshift-ci-robot removed the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Feb 1, 2019
@spadgett
Copy link
Member Author

spadgett commented Feb 1, 2019

/test all

Copy link
Member

@TheRealJon TheRealJon 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 spadgett added the lgtm Indicates that a PR is ready to be merged. label Feb 4, 2019
@spadgett
Copy link
Member Author

spadgett commented Feb 4, 2019

/hold cancel

@openshift-ci-robot openshift-ci-robot removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Feb 4, 2019
@spadgett
Copy link
Member Author

spadgett commented Feb 4, 2019

/retest

@spadgett
Copy link
Member Author

spadgett commented Feb 4, 2019

/refresh

@spadgett
Copy link
Member Author

spadgett commented Feb 4, 2019

/retest

2 similar comments
@spadgett
Copy link
Member Author

spadgett commented Feb 4, 2019

/retest

@spadgett
Copy link
Member Author

spadgett commented Feb 5, 2019

/retest

@openshift-ci-robot openshift-ci-robot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Feb 5, 2019
…nges

Change "Current Version" label to "Desired Version," which is more
accurate. Add an "Update History" table so that it's possible to see the
last completed update version.

https://bugzilla.redhat.com/show_bug.cgi?id=1666231
@openshift-ci-robot openshift-ci-robot removed lgtm Indicates that a PR is ready to be merged. needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. labels Feb 5, 2019
@spadgett
Copy link
Member Author

spadgett commented Feb 5, 2019

@TheRealJon rebased, ptal

Copy link
Member

@TheRealJon TheRealJon left a comment

Choose a reason for hiding this comment

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

/lgtm

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Feb 5, 2019
@openshift-merge-robot openshift-merge-robot merged commit 6165533 into openshift:master Feb 5, 2019
@spadgett spadgett deleted the cluster-version branch February 5, 2019 20:18
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/L Denotes a PR that changes 100-499 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants