Fixes permissions and owner for undercloud.qcow2 41/48041/1
authorTim Rozet <trozet@redhat.com>
Wed, 29 Nov 2017 22:09:41 +0000 (17:09 -0500)
committerTim Rozet <trozet@redhat.com>
Wed, 29 Nov 2017 22:09:41 +0000 (17:09 -0500)
The undercloud storage disk is copied during our deployment from
modified underloud from upstream.  We copy it as owner root with wrong
permissions.  This changes the owner to qemu and sets the permissions
the same as the other storage disks that virsh creates for overcloud
nodes.

Change-Id: Ie884a586f91be3461e4b511e224d061639f04c69
Signed-off-by: Tim Rozet <trozet@redhat.com>
apex/undercloud/undercloud.py

index 5d73dd4..14be440 100644 (file)
@@ -150,7 +150,8 @@ class Undercloud:
             if os.path.exists(dest_img):
                 os.remove(dest_img)
             shutil.copyfile(src_img, dest_img)
-
+            shutil.chown(dest_img, user='qemu', group='qemu')
+            os.chmod(dest_img, 0o0744)
         # TODO(trozet):check if resize needed right now size is 50gb
         # there is a lib called vminspect which has some dependencies and is
         # not yet available in pip.  Consider switching to this lib later.