From a492addc54691fd23ce462cd06284809a3edeaa0 Mon Sep 17 00:00:00 2001
From: Roy Nieterau <roy_nieterau@hotmail.com>
Date: Thu, 7 Sep 2023 15:55:28 +0200
Subject: [PATCH] Reduce artist-facing logs for component integration for
 Ftrack + tweak "Comment is not set" log also for Kitsu to debug level

---
 .../ftrack/plugins/publish/integrate_ftrack_api.py   | 12 ++++++------
 .../plugins/publish/integrate_ftrack_description.py  |  2 +-
 .../ftrack/plugins/publish/integrate_ftrack_note.py  |  6 +++---
 .../kitsu/plugins/publish/integrate_kitsu_note.py    |  2 +-
 4 files changed, 11 insertions(+), 11 deletions(-)

diff --git a/openpype/modules/ftrack/plugins/publish/integrate_ftrack_api.py b/openpype/modules/ftrack/plugins/publish/integrate_ftrack_api.py
index 4d474fab10..858c0bb2d6 100644
--- a/openpype/modules/ftrack/plugins/publish/integrate_ftrack_api.py
+++ b/openpype/modules/ftrack/plugins/publish/integrate_ftrack_api.py
@@ -27,8 +27,8 @@ class IntegrateFtrackApi(pyblish.api.InstancePlugin):
     def process(self, instance):
         component_list = instance.data.get("ftrackComponentsList")
         if not component_list:
-            self.log.info(
-                "Instance don't have components to integrate to Ftrack."
+            self.log.debug(
+                "Instance doesn't have components to integrate to Ftrack."
                 " Skipping."
             )
             return
@@ -37,7 +37,7 @@ class IntegrateFtrackApi(pyblish.api.InstancePlugin):
         task_entity, parent_entity = self.get_instance_entities(
             instance, context)
         if parent_entity is None:
-            self.log.info((
+            self.log.debug((
                 "Skipping ftrack integration. Instance \"{}\" does not"
                 " have specified ftrack entities."
             ).format(str(instance)))
@@ -323,7 +323,7 @@ class IntegrateFtrackApi(pyblish.api.InstancePlugin):
             "type_id": asset_type_id,
             "context_id": parent_id
         }
-        self.log.info("Created new Asset with data: {}.".format(asset_data))
+        self.log.debug("Created new Asset with data: {}.".format(asset_data))
         session.create("Asset", asset_data)
         session.commit()
         return self._query_asset(session, asset_name, asset_type_id, parent_id)
@@ -384,7 +384,7 @@ class IntegrateFtrackApi(pyblish.api.InstancePlugin):
             if comment:
                 new_asset_version_data["comment"] = comment
 
-            self.log.info("Created new AssetVersion with data {}".format(
+            self.log.debug("Created new AssetVersion with data {}".format(
                 new_asset_version_data
             ))
             session.create("AssetVersion", new_asset_version_data)
@@ -555,7 +555,7 @@ class IntegrateFtrackApi(pyblish.api.InstancePlugin):
                 location=location
             )
             data["component"] = component_entity
-            self.log.info(
+            self.log.debug(
                 (
                     "Created new Component with path: {0}, data: {1},"
                     " metadata: {2}, location: {3}"
diff --git a/openpype/modules/ftrack/plugins/publish/integrate_ftrack_description.py b/openpype/modules/ftrack/plugins/publish/integrate_ftrack_description.py
index 6ed02bc8b6..ceaff8ff54 100644
--- a/openpype/modules/ftrack/plugins/publish/integrate_ftrack_description.py
+++ b/openpype/modules/ftrack/plugins/publish/integrate_ftrack_description.py
@@ -40,7 +40,7 @@ class IntegrateFtrackDescription(pyblish.api.InstancePlugin):
 
         comment = instance.data["comment"]
         if not comment:
-            self.log.info("Comment is not set.")
+            self.log.debug("Comment is not set.")
         else:
             self.log.debug("Comment is set to `{}`".format(comment))
 
diff --git a/openpype/modules/ftrack/plugins/publish/integrate_ftrack_note.py b/openpype/modules/ftrack/plugins/publish/integrate_ftrack_note.py
index 6e82897d89..10b7932cdf 100644
--- a/openpype/modules/ftrack/plugins/publish/integrate_ftrack_note.py
+++ b/openpype/modules/ftrack/plugins/publish/integrate_ftrack_note.py
@@ -47,7 +47,7 @@ class IntegrateFtrackNote(pyblish.api.InstancePlugin):
         app_label = context.data["appLabel"]
         comment = instance.data["comment"]
         if not comment:
-            self.log.info("Comment is not set.")
+            self.log.debug("Comment is not set.")
         else:
             self.log.debug("Comment is set to `{}`".format(comment))
 
@@ -127,14 +127,14 @@ class IntegrateFtrackNote(pyblish.api.InstancePlugin):
 
             note_text = StringTemplate.format_template(template, format_data)
             if not note_text.solved:
-                self.log.warning((
+                self.log.debug((
                     "Note template require more keys then can be provided."
                     "\nTemplate: {}\nMissing values for keys:{}\nData: {}"
                 ).format(template, note_text.missing_keys, format_data))
                 continue
 
             if not note_text:
-                self.log.info((
+                self.log.debug((
                     "Note for AssetVersion {} would be empty. Skipping."
                     "\nTemplate: {}\nData: {}"
                 ).format(asset_version["id"], template, format_data))
diff --git a/openpype/modules/kitsu/plugins/publish/integrate_kitsu_note.py b/openpype/modules/kitsu/plugins/publish/integrate_kitsu_note.py
index 6e5dd056f3..b66e1f01e0 100644
--- a/openpype/modules/kitsu/plugins/publish/integrate_kitsu_note.py
+++ b/openpype/modules/kitsu/plugins/publish/integrate_kitsu_note.py
@@ -121,7 +121,7 @@ class IntegrateKitsuNote(pyblish.api.ContextPlugin):
                 publish_comment = self.format_publish_comment(instance)
 
             if not publish_comment:
-                self.log.info("Comment is not set.")
+                self.log.debug("Comment is not set.")
             else:
                 self.log.debug("Comment is `{}`".format(publish_comment))