Merge pull request #1834 from dbungert/fix-target-log-dir

shutdown: fix perms on /target/var/log/installer
canonical · Oct 10, 2023 · 8eddf7f · 8eddf7f
2 parents 7549af4 + 2d72105
commit 8eddf7f
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/subiquity/server/controllers/shutdown.py b/subiquity/server/controllers/shutdown.py
@@ -114,8 +114,8 @@ async def copy_logs_to_target(self, context):
             await self.app.command_runner.run(
                 ["cp", "-aT", "/var/log/installer", target_logs]
             )
-            # Close the permissions from group writes on the target.
-            set_log_perms(target_logs, group_write=False)
+            # explicitly setting the expected permissions on this dir
+            set_log_perms(target_logs, mode=0o770, group="adm")
 
         journal_txt = os.path.join(target_logs, "installer-journal.txt")
         try: