launchpad-reviewers team mailing list archive

[Simone Pelosi <mp+475991@xxxxxxxxxxxxxxxxxx>]

Simone Pelosi has proposed merging ~pelpsi/launchpad:update-pkgcache-log-finished-process into launchpad:master.

Commit message:
Update pkgcache process should log into info
    
If the process logs into debug we will not see important logs
inside the log file.


Requested reviews:
  Launchpad code reviewers (launchpad-reviewers)

For more details, see:
https://code.launchpad.net/~pelpsi/launchpad/+git/launchpad/+merge/475991
-- 
Your team Launchpad code reviewers is requested to review the proposed merge of ~pelpsi/launchpad:update-pkgcache-log-finished-process into launchpad:master.

diff --git a/lib/lp/soyuz/scripts/update_pkgcache.py b/lib/lp/soyuz/scripts/update_pkgcache.py
index b194e7a..7f8884a 100644
--- a/lib/lp/soyuz/scripts/update_pkgcache.py
+++ b/lib/lp/soyuz/scripts/update_pkgcache.py
@@ -88,7 +88,7 @@ class PackageCacheUpdater(LaunchpadCronScript):
             self.txn.commit()
 
     def main(self):
-        self.logger.debug("Starting the package cache update")
+        self.logger.info("Starting the package cache update")
 
         # Do the package counter and cache update for each distribution.
         distroset = getUtility(IDistributionSet)
@@ -122,4 +122,4 @@ class PackageCacheUpdater(LaunchpadCronScript):
             self.txn.commit()
             self.logger.info("%s done" % distribution.name)
 
-        self.logger.debug("Finished the package cache update")
+        self.logger.info("Finished the package cache update")