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
mgr/dashboard: disable delete on multisite #51283
Conversation
160d756
to
d7d5bff
Compare
This pull request can no longer be automatically merged: a rebase is needed and changes have to be manually resolved |
jenkins test make check |
4b642a9
to
04a34b4
Compare
jenkins test make check |
...dashboard/frontend/src/app/ceph/rgw/rgw-multisite-details/rgw-multisite-details.component.ts
Outdated
Show resolved
Hide resolved
...dashboard/frontend/src/app/ceph/rgw/rgw-multisite-details/rgw-multisite-details.component.ts
Outdated
Show resolved
Hide resolved
|
||
if (node.data.type === 'zonegroup') { | ||
if (this.zonegroups.length < 2) { | ||
this.deleteTitle = 'You can not delete the only zonegroup available'; |
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 supposed this function would only return whether it is disabled or not. Maybe the return should be an empty string when it isn't disabled and a message of why is disabled if true.
...ph/rgw/models/rgw-multisite-zone-deletion-form/rgw-multisite-zone-deletion-form.component.ts
Outdated
Show resolved
Hide resolved
...ph/rgw/models/rgw-multisite-zone-deletion-form/rgw-multisite-zone-deletion-form.component.ts
Outdated
Show resolved
Hide resolved
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.
Left some suggestions
jenkins test make check |
1 similar comment
jenkins test make check |
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.
Thank you @Pegonzal for the PR. Works as expected,tested locally. Can you please rebase the PR
Fixes: https://tracker.ceph.com/issues/59441 Signed-off-by: Pedro Gonzalez Gomez <pegonzal@redhat.com>
jenkins test dashboard |
jenkins test dashboard cephadm |
2 similar comments
jenkins test dashboard cephadm |
jenkins test dashboard cephadm |
Fixed shown pools to be deleted:
And disabled delete button on specified occasions where shouldn't be able to delete
Fixes: https://tracker.ceph.com/issues/59441
Signed-off-by: Pedro Gonzalez Gomez pegonzal@redhat.com
Contribution Guidelines
To sign and title your commits, please refer to Submitting Patches to Ceph.
If you are submitting a fix for a stable branch (e.g. "pacific"), please refer to Submitting Patches to Ceph - Backports for the proper workflow.
Checklist
Show available Jenkins commands
jenkins retest this please
jenkins test classic perf
jenkins test crimson perf
jenkins test signed
jenkins test make check
jenkins test make check arm64
jenkins test submodules
jenkins test dashboard
jenkins test dashboard cephadm
jenkins test api
jenkins test docs
jenkins render docs
jenkins test ceph-volume all
jenkins test ceph-volume tox
jenkins test windows