launchpad-reviewers team mailing list archive

[İlker Emre Koç <mp+483566@xxxxxxxxxxxxxxxxxx>]

İlker Emre Koç has proposed merging ~ilkeremrekoc/launchpad:publisher-log-level-info into launchpad:master.

Commit message:
change ignore pfoe log level to info

Requested reviews:
  Launchpad code reviewers (launchpad-reviewers)

For more details, see:
https://code.launchpad.net/~ilkeremrekoc/launchpad/+git/launchpad/+merge/483566

We cannot see the logs of IgnorePoolFileOverwriteError right now since
the diskpool configs (which artifactory inherits) sets the log level
to INFO while we currently try to log it into DEBUG. Thus, we change
the log level of the log from DEBUG to INFO.
-- 
Your team Launchpad code reviewers is requested to review the proposed merge of ~ilkeremrekoc/launchpad:publisher-log-level-info into launchpad:master.

diff --git a/lib/lp/archivepublisher/artifactory.py b/lib/lp/archivepublisher/artifactory.py
index 8de1a48..1925ba3 100644
--- a/lib/lp/archivepublisher/artifactory.py
+++ b/lib/lp/archivepublisher/artifactory.py
@@ -379,7 +379,7 @@ class ArtifactoryPoolEntry:
                     self.archive.repository_format
                     in IGNORE_PFOES_REPOSITORY_TYPES
                 ):
-                    self.logger.debug(
+                    self.logger.info(
                         f"Ignoring PoolFileOverwriteError: {error_message} "
                         "as it is a known limitation for "
                         f"{self.archive.repository_format.title} packages."