Fix issue 'Permission denied' due to repo permissions on redhat linux

Author: fabi200123

coriolis/osmorphing/redhat.py

@@ -270,7 +270,12 @@ def _find_yum_repos(self, repos_to_enable=[]):
         installed_repos = []
         for file in repofiles:
             path = os.path.join(reposdir_path, file)
-            content = self._read_file(path).decode()
+            try:
+                content = self._read_file_sudo(path).decode()
+            except Exception as e:
+                LOG.warning(
+                    "Could not read yum repository file %s: %s", path, e)
+                continue
             for line in content.splitlines():
                 m = re.match(r'^\[(.+)\]$', line)
                 if m:

coriolis/tests/osmorphing/test_redhat.py

@@ -453,26 +453,27 @@ def test__yum_clean_all_path_not_exists(self, mock_test_path,
         mock_exec_cmd_chroot.assert_called_once_with("yum clean all")
 
     @mock.patch.object(base.BaseLinuxOSMorphingTools, '_list_dir')
-    @mock.patch.object(base.BaseLinuxOSMorphingTools, '_read_file')
-    def test__find_yum_repos_found(self, mock_read_file, mock_list_dir):
+    @mock.patch.object(base.BaseLinuxOSMorphingTools, '_read_file_sudo')
+    def test__find_yum_repos_found(self, mock_read_file_sudo, mock_list_dir):
         mock_list_dir.return_value = ['file1.repo', 'file2.repo']
-        mock_read_file.return_value = b'[repo1]\n[repo2]'
+        mock_read_file_sudo.return_value = b'[repo1]\n[repo2]'
         repos_to_enable = ['repo1']
 
         result = self.morphing_tools._find_yum_repos(repos_to_enable)
 
-        mock_read_file.assert_has_calls([
+        mock_read_file_sudo.assert_has_calls([
             mock.call('etc/yum.repos.d/file1.repo'),
             mock.call('etc/yum.repos.d/file2.repo')
         ])
 
         self.assertEqual(result, ['repo1'])
 
     @mock.patch.object(base.BaseLinuxOSMorphingTools, '_list_dir')
-    @mock.patch.object(base.BaseLinuxOSMorphingTools, '_read_file')
-    def test__find_yum_repos_not_found(self, mock_read_file, mock_list_dir):
+    @mock.patch.object(base.BaseLinuxOSMorphingTools, '_read_file_sudo')
+    def test__find_yum_repos_not_found(self, mock_read_file_sudo,
+                                       mock_list_dir):
         mock_list_dir.return_value = ['file1.repo', 'file2.repo']
-        mock_read_file.return_value = b'[repo1]\n[repo2]'
+        mock_read_file_sudo.return_value = b'[repo1]\n[repo2]'
         repos_to_enable = ['repo3']
 
         with self.assertLogs('coriolis.osmorphing.redhat', level=logging.WARN):