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.

Sign up for GitHub

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

Jump to bottom

Wrong namespaces deleted on deleting a subsystem #1063

Closed
gbregman opened this issue Jan 28, 2025 · 0 comments · Fixed by #1064
Closed

Wrong namespaces deleted on deleting a subsystem #1063

gbregman opened this issue Jan 28, 2025 · 0 comments · Fixed by #1064
Assignees
@gbregman

Comments

@gbregman
Copy link
Collaborator

gbregman commented Jan 28, 2025 •
edited
Loading

I created two subsystems with NQNs nqn.2016-06.io.spdk:cnode1 and nqn.2016-06.io.spdk:cnode10. I created two namespaces on each subsystem. When I deleted subsystem nqn.2016-06.io.spdk:cnode1, all the namespaces got deleted instead of just the two belonging to the deleted subsystem.

See BZ #2342506 and BZ #2342548
@github-project-automation github-project-automation bot moved this to 🆕 New in NVMe-oF Jan 28, 2025
@gbregman gbregman self-assigned this Jan 28, 2025
gbregman added a commit to gbregman/ceph-nvmeof that referenced this issue Jan 28, 2025
@gbregman
Don't delete other subsys namespace when deleting subsys.
17156a1 
Fixes ceph#1063

Signed-off-by: Gil Bregman <[email protected]>
gbregman added a commit to gbregman/ceph-nvmeof that referenced this issue Jan 28, 2025
@gbregman
Don't delete other subsys namespace when deleting subsys.
55e29f7 
Fixes ceph#1063

Signed-off-by: Gil Bregman <[email protected]>
gbregman added a commit to gbregman/ceph-nvmeof that referenced this issue Jan 28, 2025
@gbregman
Don't delete other subsys namespace when deleting subsys.
7431c18 
Fixes ceph#1063

Signed-off-by: Gil Bregman <[email protected]>
@gbregman gbregman mentioned this issue Jan 28, 2025
Merged
@github-project-automation github-project-automation bot moved this from 🆕 New to ✅ Done in NVMe-oF Jan 29, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@gbregman gbregman

Labels
None yet
Projects
NVMe-oF
Archived in project
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Don't delete other subsys namespace when deleting a subsys gbregman/ceph-nvmeof
1 participant
@gbregman