Restructure log location

Resolves #7 (closed)

mpsd-software-environment.py

@@ -151,21 +151,20 @@ def read_metadata_from_logfile(logfile: Union[str, Path]) -> dict:
     }
 
 
-def get_installer_log_file(args: argparse.Namespace, script_dir: str) -> str:
+def get_installer_log_file(mpsd_release: str, cmd: str, script_dir: str) -> str:
     """Get installer log file name."""
     # Get machine configs
     os.environ.get("MPSD_OS", "UNKNOWN_OS")
     mpsd_microarch = get_native_microarchitecture()
-    mpsd_release = args.release
     # parse logging first
     # decide the log_file_name
     installer_log_name, build_log_name = create_log_file_names(
-        mpsd_release=mpsd_release, mpsd_microarch=mpsd_microarch, action=args.action
+        mpsd_release=mpsd_release, mpsd_microarch=mpsd_microarch, action=cmd
     )
-    log_folder = script_dir / mpsd_release / mpsd_microarch / "logs"
+    log_folder = script_dir / mpsd_release / "logs"
     # if the log_folder dosent exist, dont log this message if
     #  the command is a info-only command
-    if args.action not in ["status", "available"]:
+    if cmd not in ["status", "available"]:
         if not os.path.exists(log_folder):
             os.makedirs(log_folder)
         installer_log_file = log_folder / installer_log_name
@@ -309,7 +308,7 @@ def set_up_logging(loglevel="warning", filename=None):
     # short message
     #
     logging.debug(
-        f"Logging has been setup, loglevel={loglevel.upper()}"
+        f"Logging has been setup, loglevel={loglevel.upper()} "
         + f"{filename=} {rich_available=}"
     )
 
@@ -801,8 +800,8 @@ def install_environment(
                 mpsd_release,
                 script_dir,
                 msg=(
-                    f"CMD: bash {spack_setup_script} {' '.join(install_flags)}"
-                    "{toolchain}"
+                    f"CMD: bash {spack_setup_script} {' '.join(install_flags)} "
+                    f"{toolchain}"
                 ),
             )
             run(
@@ -910,9 +909,10 @@ def main():
 
     # target dir is the place where this script exists. the
     script_dir = Path(os.path.dirname(os.path.realpath(__file__)))
-    installer_log_file = get_installer_log_file(args, script_dir)
 
-    set_up_logging(args.loglevel, installer_log_file)
+    set_up_logging(
+        args.loglevel, get_installer_log_file(args.release, args.action, script_dir)
+    )
 
     # Check the command and run related function
     if args.action == "remove":