Merge pull request #1835 from dbungert/target-log-dir-mantic

shutdown: fix perms on /target/var/log/installer
canonical · Oct 10, 2023 · 01fd8d2 · 01fd8d2
2 parents 7717b0d + 5950460
commit 01fd8d2
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: