chore: add GitHub download logging for visibility into corner cases

In serious cases where there are problems with the GitHub download there
can be unexplained failures, as such situations are not caught due to
the use of a CLI command using Python's subproces. Add some logging
until the GitHub download is refactored to use a Python client and have
better error-handling.

Author: schivmeister

src/ted_sws/mapping_suite_processor/adapters/github_ms_project_downloader.py

@@ -5,7 +5,7 @@
 import tempfile
 from typing import ClassVar
 from src.ted_sws import config
-
+from src.ted_sws.event_manager.services.log import log_technical_info
 # TODO: get from env or config
 MAPPINGS_DIR_NAME = "mappings"
 MS_CONFIG_DIR_NAME = "config"
@@ -77,11 +77,15 @@ def get_git_head_hash(git_repository_path: pathlib.Path) -> str:
         with tempfile.TemporaryDirectory() as tmp_dir:
             temp_dir_path = pathlib.Path(tmp_dir)
             bash_script = f"cd {temp_dir_path} && git clone --branch {self.branch_or_tag_name} {self.github_repository_url}"
-            subprocess.run(bash_script, shell=True,
-                           stdout=subprocess.DEVNULL,
-                           stderr=subprocess.STDOUT)
+            result = subprocess.run(bash_script, shell=True,
+                                    capture_output=True, text=True)
+            log_technical_info(
+                message=f"Downloaded stdout '{result.stdout}'")
+            log_technical_info(
+                message=f"Downloaded stderr '{result.stderr}'")
             git_last_commit_hash = get_git_head_hash(
                 git_repository_path=temp_dir_path / self.repository_name)
             downloaded_tmp_project_path = temp_dir_path / self.repository_name
             shutil.copytree(downloaded_tmp_project_path, output_project_path, dirs_exist_ok=True)
+
         return git_last_commit_hash

src/ted_sws/mapping_suite_processor/services/mapping_package_processor.py

@@ -101,8 +101,6 @@ def load_mapping_suite_and_packages_from_github_to_mongo_db(mongodb_client: Mong
     """
     branch_or_tag_name = branch_or_tag_name if branch_or_tag_name else DEFAULT_BRANCH_NAME
     github_repository_url = github_repository_url if github_repository_url else config.GITHUB_TED_SWS_ARTEFACTS_URL
-    log_technical_info(
-        message=f"Downloading mapping suite from GitHub repository '{github_repository_url}' on branch/tag '{branch_or_tag_name}'")
     mapping_package_downloader = GitHubMappingSuiteDownloader(
         github_repository_url=github_repository_url, branch_or_tag_name=branch_or_tag_name)
     mappings_dir_name = mapping_package_downloader.MAPPINGS_DIR_NAME
@@ -116,6 +114,8 @@ def load_mapping_suite_and_packages_from_github_to_mongo_db(mongodb_client: Mong
         mappings_dir_path = tmp_dir_path / mappings_dir_name
         ms_config_dir_path = tmp_dir_path / ms_config_dir_name
         ms_config_file_path = ms_config_dir_path / ms_config_file_name
+        log_technical_info(
+            message=f"Downloading mapping suite from GitHub repository '{github_repository_url}' on branch/tag '{branch_or_tag_name}'")
         git_last_commit_hash = mapping_package_downloader.download(output_project_path=tmp_dir_path)
 
         # load project config if available