-
Notifications
You must be signed in to change notification settings - Fork 45
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
ui/volume: rephrase the volume status from Available to Ready #2248
ui/volume: rephrase the volume status from Available to Ready #2248
Conversation
Hello chengyanjin,My role is to assist you with the merge of this Status report is not available. |
Integration data createdI have created the integration data for the additional destination branches.
The following branches will NOT be impacted:
You can set option
|
Waiting for approvalThe following approvals are needed before I can proceed with the merge:
Peer approvals must include at least 1 approval from the following list:
|
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.
I think you forgot to update the code for the hint popup in src/containers/NodeVolumes.js
: it's still using STATUS_AVAILABLE
instead of STATUS_READY
Waiting for approvalThe following approvals are needed before I can proceed with the merge:
Peer approvals must include at least 1 approval from the following list:
The following reviewers are expecting changes from the author, or must review again: |
45e9933
to
b4018f3
Compare
History mismatchMerge commit #c77366ba6fd129d12b0e22b9502af4175e4dc12c on the integration branch It is likely due to a rebase of the branch Please use the |
b4018f3
to
0195edd
Compare
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.
Works as expected
2 minor comments + 1 remark on a commit that wasn't correctly split.
e438476
to
5489868
Compare
The volume status available is confusing. For example, when the PV is bound, we displayed Available Refs: #2245
should return Ready as status when Ready condition is True Refs: #2245
update the indentation to meet the setting for vscode Refs: #2245
remove the single quote to meet the format of vscode Refs: #2245
5489868
to
bb95149
Compare
- since we update the volume status from availble to ready, we want to update the cypress test to check if the volume status is ready Refs: #2245
/approve |
History mismatchMerge commit #c77366ba6fd129d12b0e22b9502af4175e4dc12c on the integration branch It is likely due to a rebase of the branch Please use the The following options are set: approve |
/reset |
Reset completeI have successfully deleted this pull request's integration branches. The following options are set: approve |
Integration data createdI have created the integration data for the additional destination branches.
The following branches will NOT be impacted:
You can set option
The following options are set: approve |
Build failedThe build for commit did not succeed in branch w/2.6/bugfix/2245-rephrase-the-volume-status-when-the-pv-is-bound. The following options are set: approve |
In the queueThe changeset has received all authorizations and has been added to the The changeset will be merged in:
The following branches will NOT be impacted:
There is no action required on your side. You will be notified here once IMPORTANT Please do not attempt to modify this pull request.
If you need this pull request to be removed from the queue, please contact a The following options are set: approve |
I have successfully merged the changeset of this pull request
The following branches have NOT changed:
Please check the status of the associated issue None. Goodbye chengyanjin. |
Component: ui, volume
Context:
The volume status available is confusing. For example, when the PV is bound, we displayed
Available
.Summary:
Rephrase the volume status from
Available
toReady
Acceptance criteria:
Closes: #2245