launchpad-reviewers team mailing list archive

[Simone Pelosi <mp+480455@xxxxxxxxxxxxxxxxxx>]

Simone Pelosi has proposed merging ~pelpsi/launchpad:add-snap-components-logs into launchpad:master.

Commit message:
Add logs to SnapStoreUploadJob for Snap and Snap Components

Requested reviews:
  Launchpad code reviewers (launchpad-reviewers)

For more details, see:
https://code.launchpad.net/~pelpsi/launchpad/+git/launchpad/+merge/480455
-- 
Your team Launchpad code reviewers is requested to review the proposed merge of ~pelpsi/launchpad:add-snap-components-logs into launchpad:master.

diff --git a/lib/lp/snappy/model/snapbuildjob.py b/lib/lp/snappy/model/snapbuildjob.py
index 9a73f64..3a326bd 100644
--- a/lib/lp/snappy/model/snapbuildjob.py
+++ b/lib/lp/snappy/model/snapbuildjob.py
@@ -409,6 +409,15 @@ class SnapStoreUploadJob(SnapBuildJobDerived):
                         )
                         self.attempt_count = 1
                 if "status_url" not in self.store_metadata:
+                    self.logger.debug(
+                        "[SnapStoreUploadJob] Pushing build %s with id %s.",
+                        self.snapbuild.snap.name,
+                        self.snapbuild.id,
+                    )
+                    self.logger.debug(
+                        "[SnapStoreUploadJob] Components: %s",
+                        self.components_ids,
+                    )
                     self.status_url = client.push(
                         self.snapbuild, self.upload_id, self.components_ids
                     )