]> git.apps.os.sepia.ceph.com Git - ceph.git/commitdiff
ceph_disk: allow "no fsid" on activate 18991/head
authorDan Mick <dan.mick@redhat.com>
Wed, 29 Nov 2017 23:29:17 +0000 (15:29 -0800)
committerDan Mick <dan.mick@redhat.com>
Wed, 29 Nov 2017 23:29:17 +0000 (15:29 -0800)
The intent was to allow "no fsid" configurations when only one
conf file named 'ceph.conf' was present, but the code has a bug
in that ceph-osd --show-config-value will return a default all-0
uuid.  Treat 'all-0' as we were treating 'None'.

Signed-off-by: Dan Mick <dan.mick@redhat.com>
src/ceph-disk/ceph_disk/main.py

index 798072ce8a79d707f8f8f376f14c9fb82572a113..3e8b62f9cab8f42db05b09c3803aa58301d863ea 100644 (file)
@@ -1258,7 +1258,8 @@ def get_fsid(cluster):
     :return: The fsid or raises Error.
     """
     fsid = get_conf_with_default(cluster=cluster, variable='fsid')
-    if fsid is None:
+    # uuids from boost always default to 'the empty uuid'
+    if fsid == '00000000-0000-0000-0000-000000000000':
         raise Error('getting cluster uuid from configuration failed')
     return fsid.lower()