Set permissions after running mic appliance

Due to the bug in Plan9 driver or in qemu permissions of files and
directories on plan9 share are set only for owner(jenkins in our case).
This makes mic/output/* unreadable after directory is rsynced to
download server. Apache raises 403 Forbidden error when
post-image-creation job tries to access build results.

Fixes: #1206

Change-Id: Id723d7323452c7d6ea4df1f131f1498351a78eb2
Signed-off-by: Ed Bartosh <eduard.bartosh@intel.com>
Reviewed-on: https://otctools.jf.intel.com/review/6439
Tested-by: OTC Tools Tester <ed.bartosh@linux.intel.com>
Reviewed-by: Hasan Wan <hasan.wan@intel.com>

diff --git a/common/utils.py b/common/utils.py
index 9050af6..2f33671 100644 (file)
--- a/common/utils.py
+++ b/common/utils.py
@@ -120,3 +120,15 @@ def sync(source, destination):
         raise RuntimeException("Execution of %s failed: %s" % (cmd, str(err)))
 
     return ret_code
+
+def set_permissions(tpath, modes=(0644, 0755)):
+    """
+    Recursively set permission bits for files and directories.
+    Parameters:
+        tpath: top directory path
+        modes: sequence of two modes: (mode_for_files, mode_for_directories)
+    """
+    for root, dirs, files in os.walk(tpath):
+        for fname in files + dirs:
+            path = os.path.join(root, fname)
+            os.chmod(path, os.stat(path).st_mode | modes[fname in dirs])

diff --git a/job_imager.py b/job_imager.py
index f4d63b8..c4b48c2 100755 (executable)
--- a/job_imager.py
+++ b/job_imager.py
@@ -8,7 +8,7 @@ import shutil
 import subprocess
 
 from common.buildtrigger import trigger_info, trigger_next
-from common.utils import sync
+from common.utils import sync, set_permissions
 
 def get_xml(image_dir, ks_name):
     """ Get the xml the image, return the body as string """
@@ -91,6 +91,8 @@ def main():
         print 'starting mic inside VM to create image'
         ret = run_inside_vm(vm_image, os.getenv("VM_MEMORY", 8192),
                             os.getenv("VM_CPUS", 8), basedir)
+        # workaround for qemu/9p bug in mapping permissions
+        set_permissions(outdir, (0644, 0755))
     else:
         log = os.path.join(outdir, '%s_%s.log.txt' % (build_id, name))
         cache = os.path.join(basedir, 'cache')