From 1bbab5c80dd350c1f5fe3fc14b2838a114f7dd9c Mon Sep 17 00:00:00 2001
From: Ayaz Salikhov <mathbunnyru@gmail.com>
Date: Fri, 21 Feb 2025 22:24:59 +0000
Subject: [PATCH] Add more logs in tagging

---
 tagging/apps/merge_tags.py      |  1 +
 tagging/apps/write_manifest.py  | 10 +++++++---
 tagging/apps/write_tags_file.py |  6 ++++--
 3 files changed, 12 insertions(+), 5 deletions(-)

diff --git a/tagging/apps/merge_tags.py b/tagging/apps/merge_tags.py
index 2352a5a556..d652655acd 100755
--- a/tagging/apps/merge_tags.py
+++ b/tagging/apps/merge_tags.py
@@ -56,6 +56,7 @@ def merge_tags(
         LOGGER.info(f"Found images: {existing_images}")
         docker["manifest", "create", tag][existing_images] & plumbum.FG
         docker["manifest", "push", tag] & plumbum.FG
+        LOGGER.info(f"Successfully merged and pushed tag: {tag}")
 
 
 if __name__ == "__main__":
diff --git a/tagging/apps/write_manifest.py b/tagging/apps/write_manifest.py
index e25d9ac4cc..0f68d97d64 100755
--- a/tagging/apps/write_manifest.py
+++ b/tagging/apps/write_manifest.py
@@ -50,7 +50,9 @@ def write_build_history_line(
     )
     build_history_line = f"| {date_column} | {image_column} | {links_column} |"
     hist_lines_dir.mkdir(parents=True, exist_ok=True)
-    (hist_lines_dir / f"{filename}.txt").write_text(build_history_line)
+    file = hist_lines_dir / f"{filename}.txt"
+    file.write_text(build_history_line)
+    LOGGER.info(f"Build history line written to: {file}")
 
 
 def write_manifest_file(
@@ -79,7 +81,9 @@ def write_manifest_file(
     markdown_content = "\n\n".join(markdown_pieces) + "\n"
 
     manifests_dir.mkdir(parents=True, exist_ok=True)
-    (manifests_dir / f"{filename}.md").write_text(markdown_content)
+    file = manifests_dir / f"{filename}.md"
+    file.write_text(markdown_content)
+    LOGGER.info(f"Manifest file written to: {file}")
 
 
 def write_manifest(
@@ -92,7 +96,7 @@ def write_manifest(
     manifests_dir: Path,
     repository: str,
 ) -> None:
-    LOGGER.info(f"Creating manifests for image: {short_image_name}")
+    LOGGER.info(f"Creating manifests for image: {registry}/{owner}/{short_image_name}")
     taggers, manifests = get_taggers_and_manifests(short_image_name)
 
     image = f"{registry}/{owner}/{short_image_name}:latest"
diff --git a/tagging/apps/write_tags_file.py b/tagging/apps/write_tags_file.py
index 8192ba6e91..88851f415d 100755
--- a/tagging/apps/write_tags_file.py
+++ b/tagging/apps/write_tags_file.py
@@ -25,7 +25,7 @@ def write_tags_file(
     """
     Writes tags file for the image <registry>/<owner>/<short_image_name>:latest
     """
-    LOGGER.info(f"Tagging image: {short_image_name}")
+    LOGGER.info(f"Tagging image: {registry}/{owner}/{short_image_name}")
     taggers, _ = get_taggers_and_manifests(short_image_name)
 
     image = f"{registry}/{owner}/{short_image_name}:latest"
@@ -45,7 +45,9 @@ def write_tags_file(
                 f"{registry}/{owner}/{short_image_name}:{tags_prefix}-{tag_value}"
             )
     tags_dir.mkdir(parents=True, exist_ok=True)
-    (tags_dir / filename).write_text("\n".join(tags))
+    file = tags_dir / filename
+    file.write_text("\n".join(tags))
+    LOGGER.info(f"Tags file written to: {file}")
 
 
 if __name__ == "__main__":