From 04a3cb8cbad9b1700286c5de67aa5d974cf54400 Mon Sep 17 00:00:00 2001
From: Elena Taivan <elena.taivan@windriver.com>
Date: Wed, 29 Apr 2020 08:44:13 +0000
Subject: [PATCH] Changing permissions for conversion folder

Adding writing permissions to '/opt/conversion' mountpoint
so openstack image conversion can happen there.

Change-Id: Id1a91db6570dcbed3b8068e79e72f5bb800f24ad
Partial-bug: 1819688
Signed-off-by: Elena Taivan <elena.taivan@windriver.com>
---
 .../src/modules/platform/manifests/filesystem.pp   | 14 ++++++++++++--
 1 file changed, 12 insertions(+), 2 deletions(-)

diff --git a/puppet-manifests/src/modules/platform/manifests/filesystem.pp b/puppet-manifests/src/modules/platform/manifests/filesystem.pp
index c798b562..1b5ac8d4 100644
--- a/puppet-manifests/src/modules/platform/manifests/filesystem.pp
+++ b/puppet-manifests/src/modules/platform/manifests/filesystem.pp
@@ -41,6 +41,10 @@ define platform::filesystem (
       options => 'defaults',
       fstype  => $fs_type,
     }
+    -> file { $mountpoint:
+      ensure => $ensure,
+      force  => true,
+    }
     -> exec { "wipe start of device ${device}":
       command => "dd if=/dev/zero of=${device} bs=512 count=34",
       onlyif  => "blkid ${device}",
@@ -95,6 +99,9 @@ define platform::filesystem (
       command => "mount ${mountpoint}",
       path    => '/usr/bin'
     }
+    -> exec {"Change ${mountpoint} dir permissions":
+      command => "chmod ${mode} ${mountpoint}",
+    }
   }
 }
 
@@ -161,7 +168,8 @@ class platform::filesystem::conversion::params (
   $mountpoint = '/opt/conversion',
   $devmapper = '/dev/mapper/cgts--vg-conversion--lv',
   $fs_type = 'ext4',
-  $fs_options = ' '
+  $fs_options = ' ',
+  $mode = '0750'
 ) { }
 
 class platform::filesystem::scratch::params (
@@ -190,6 +198,7 @@ class platform::filesystem::conversion
 
   if $conversion_enabled {
     $ensure = present
+    $mode = '0777'
   }
   platform::filesystem { $lv_name:
     ensure     => $ensure,
@@ -197,7 +206,8 @@ class platform::filesystem::conversion
     lv_size    => $lv_size,
     mountpoint => $mountpoint,
     fs_type    => $fs_type,
-    fs_options => $fs_options
+    fs_options => $fs_options,
+    mode       => $mode
   }
 }