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

reef: mgr/dashboard: Create subvol of same name in different group #55369

Merged
merged 1 commit into from
Feb 8, 2024

Conversation

afreen23
Copy link
Contributor

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


backport of #55349
parent tracker: https://tracker.ceph.com/issues/64112

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

Fixes https://tracker.ceph.com/issues/64112

Issue:
Currently, we are unable to create subvolume of same name in different
subvolume group

Fix:
We are validating only the filesystem name of subvolume
which is stopping the creation a subvolume of same name.
Added more granularity , by adding the subvolumegroup name.

Signed-off-by: Afreen <afreen23.git@gmail.com>
(cherry picked from commit fb79df4)
@afreen23 afreen23 requested a review from a team as a code owner January 30, 2024 10:39
@afreen23 afreen23 requested review from aaSharma14 and Pegonzal and removed request for a team January 30, 2024 10:39
@afreen23 afreen23 added this to the reef milestone Jan 30, 2024
@nizamial09
Copy link
Member

I just remembered that this alone might not be sufficient to fix the issue in reef. We will need the modification done in this file, which I'll soon backport to reef. Once that is backported, we can proceed with this PR.

@nizamial09 nizamial09 added the DNM label Jan 30, 2024
@nizamial09
Copy link
Member

you won't believe this but I already backported everything 3 weeks ago: #55186 and then I forgot about doing it ever. So I blocked this PR for no reason at all :)

@nizamial09
Copy link
Member

jenkins test api

@nizamial09 nizamial09 removed the DNM label Feb 8, 2024
@afreen23
Copy link
Contributor Author

afreen23 commented Feb 8, 2024

jenkins test api

@afreen23
Copy link
Contributor Author

afreen23 commented Feb 8, 2024

jenkins test dashboard

@afreen23
Copy link
Contributor Author

afreen23 commented Feb 8, 2024

jenkins test dashboard cephadm

1 similar comment
@afreen23
Copy link
Contributor Author

afreen23 commented Feb 8, 2024

jenkins test dashboard cephadm

@nizamial09 nizamial09 merged commit fe87698 into ceph:reef Feb 8, 2024
11 of 13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
2 participants