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

pacific: mgr/dashboard: fix nfs exports form issues with squash field #47960

Merged
merged 2 commits into from
Oct 11, 2022

Conversation

nizamial09
Copy link
Member

backport tracker: https://tracker.ceph.com/issues/57435


backport of #47571
parent tracker: https://tracker.ceph.com/issues/57114

this backport was staged using ceph-backport.sh version 16.0.0.6848
find the latest version at https://github.com/ceph/ceph/blob/main/src/script/ceph-backport.sh

1. Squash is not a mandatory field.
2. If the export is created from backend, then squash can have many
   different names for each types of squash. for eg.
```
['root', 'root_squash', 'rootsquash', 'rootid', 'root_id_squash', 'rootidsquash', 'all', 'all_squash', 'allsquash', 'all_anomnymous', 'allanonymous', 'no_root_squash', 'none', 'noidsquash']
```
so we need a proper matching in the ui too otherwise the edit field will
not have any value for the squash field.

3. Removed unncessary dropdown helper if there are values for the squash
   and access types.

Fixes: https://tracker.ceph.com/issues/57114
Signed-off-by: Nizamudeen A <nia@redhat.com>
(cherry picked from commit 23c00ed)
This actually broke the doc service when the developer environment is of
main branch

Signed-off-by: Nizamudeen A <nia@redhat.com>
(cherry picked from commit be5ab2f)
@nizamial09 nizamial09 requested a review from a team as a code owner September 4, 2022 07:42
@nizamial09 nizamial09 requested review from avanthakkar and pereman2 and removed request for a team September 4, 2022 07:42
@nizamial09 nizamial09 added this to the pacific milestone Sep 4, 2022
@github-actions github-actions bot added the pybind label Sep 4, 2022
@nizamial09
Copy link
Member Author

jenkins test make check

@nizamial09
Copy link
Member Author

jenkins test api

@nizamial09 nizamial09 merged commit 1fd46cc into ceph:pacific Oct 11, 2022
@nizamial09 nizamial09 deleted the wip-57435-pacific branch October 11, 2022 04:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants