From: Alfredo Deza Date: Mon, 10 Dec 2018 18:27:20 +0000 (-0500) Subject: ceph-volume lvm.activate always chown osd path before osd-prime-dir in bluestore X-Git-Tag: v13.2.5~141^2~6 X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=7e259988272762206016368215d7b1cc2da1d55f;p=ceph.git ceph-volume lvm.activate always chown osd path before osd-prime-dir in bluestore Signed-off-by: Alfredo Deza (cherry picked from commit 0af31ec24533091aa4991b2343fe80da94777b4d) --- diff --git a/src/ceph-volume/ceph_volume/devices/lvm/activate.py b/src/ceph-volume/ceph_volume/devices/lvm/activate.py index 852c314c2993..f85747fe438d 100644 --- a/src/ceph-volume/ceph_volume/devices/lvm/activate.py +++ b/src/ceph-volume/ceph_volume/devices/lvm/activate.py @@ -151,7 +151,10 @@ def activate_bluestore(lvs, no_systemd=False): db_device_path = get_osd_device_path(osd_lv, lvs, 'db', dmcrypt_secret=dmcrypt_secret) wal_device_path = get_osd_device_path(osd_lv, lvs, 'wal', dmcrypt_secret=dmcrypt_secret) - # Once symlinks are removed, the osd dir can be 'primed again. + # Once symlinks are removed, the osd dir can be 'primed again. chown first, + # regardless of what currently exists so that ``prime-osd-dir`` can succeed + # even if permissions are somehow messed up + system.chown(osd_path) prime_command = [ 'ceph-bluestore-tool', '--cluster=%s' % conf.cluster, 'prime-osd-dir', '--dev', osd_lv_path,