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

mon: detect existing fs and duplicate name earlier #13471

Merged
merged 1 commit into from Feb 22, 2017

Conversation

Projects
None yet
2 participants
@batrick
Member

batrick commented Feb 17, 2017

Detecting creation of an equivalent fs was made impossible by
052c3d3, as an existing file system
will obviously have objects in its pool. This commit moves the pool with
objects check later and uses a more helpful errno (EEXIST) for the case
where a fs with the same name already exists (but uses different pools).

Fixes: http://tracker.ceph.com/issues/18964

Signed-off-by: Patrick Donnelly pdonnell@redhat.com

mon: detect existing fs and duplicate name earlier
Detecting creation of an equivalent fs was made impossible by
052c3d3, as an existing file system
will obviously have objects in its pool. This commit moves the pool with
objects check later and uses a more helpful errno (EEXIST) for the case
where a fs with the same name already exists (but uses different pools).

Fixes: http://tracker.ceph.com/issues/18964

Signed-off-by: Patrick Donnelly <pdonnell@redhat.com>
@jcsp

jcsp approved these changes Feb 20, 2017

@jcsp jcsp merged commit 0d817a2 into ceph:master Feb 22, 2017

3 checks passed

Signed-off-by all commits in this PR are signed
Details
Unmodifed Submodules submodules for project are unmodified
Details
default Build finished.
Details

@batrick batrick deleted the batrick:fs-new-eexist branch Feb 27, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment