add more

Signed-off-by: Peter Zhu <[email protected]>

src/test_workflow/integ_test/distribution.py

@@ -41,6 +41,13 @@ def config_path(self) -> str:
         """
         pass
 
+    @property
+    def log_dir(self) -> str:
+        """
+        Return the log directory for the distribution
+        """
+        pass
+
     @abstractmethod
     def install(self, bundle_name: str) -> None:
         """

src/test_workflow/integ_test/distribution_deb.py

@@ -25,6 +25,10 @@ def install_dir(self) -> str:
     def config_path(self) -> str:
         return os.path.join(os.sep, "etc", self.filename, self.config_filename)
 
+    @property
+    def log_dir(self) -> str:
+        return os.path.join(os.sep, "var", "logs", self.filename)
+
     def install(self, bundle_name: str) -> None:
         logging.info(f"Installing {bundle_name} in {self.install_dir}")
         logging.info("deb installation requires sudo, script will exit if current user does not have sudo access")
@@ -44,7 +48,7 @@ def install(self, bundle_name: str) -> None:
                 '&&',
                 f'sudo chmod 0666 {self.config_path}',
                 '&&',
-                f'sudo chmod 0755 {os.path.dirname(self.config_path)}'
+                f'sudo chmod 0755 {os.path.dirname(self.config_path)} {self.log_dir}'
             ]
         )
         subprocess.check_call(deb_install_cmd, cwd=self.work_dir, shell=True)