Merge hexr:production-debug-1867836 into hexr:master

Proposed by Sheila Miguez
Status: Merged
Approved by: Sheila Miguez
Approved revision: 9f2bde5f71d3aa0419a4ca73ba7a69ed096785a9
Merged at revision: b28b391ba6b882268a642960d8dfcc1c334d7d73
Proposed branch: hexr:production-debug-1867836
Merge into: hexr:master
Diff against target: 35 lines (+0/-4)
2 files modified
apps/uploads/tasks.py (+0/-2)
certification/cert_api/resources.py (+0/-2)
Reviewer Review Type Date Requested Status
HEXR Developers Pending
Review via email: mp+380841@code.launchpad.net

Commit message

Remove debug loglines

This removes loglines that were added while
investigating LP: #1867836

Description of the change

see commit

To post a comment you must log in.
Revision history for this message
Sheila Miguez (codersquid) wrote :

Self approve so that I can deplooy this asap

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
diff --git a/apps/uploads/tasks.py b/apps/uploads/tasks.py
index f4eac87..4bb76a1 100644
--- a/apps/uploads/tasks.py
+++ b/apps/uploads/tasks.py
@@ -57,10 +57,8 @@ def create_submission_report_task(submission_id,
57 raise Exception(msg)57 raise Exception(msg)
5858
59 # Now that a report is created, save the service id from the submission59 # Now that a report is created, save the service id from the submission
60 logger.error("DEBUG submission_id for hardware %s before save %s" % (hardware.canonical_id, submission_id))
61 report.submission_id = submission_id60 report.submission_id = submission_id
62 report.save()61 report.save()
63 logger.error("DEBUG submission_id for hardware %s after save %s" % (hardware.canonical_id, submission_id))
6462
65 # Denormalize the report63 # Denormalize the report
66 try:64 try:
diff --git a/certification/cert_api/resources.py b/certification/cert_api/resources.py
index 4fe474c..4966095 100644
--- a/certification/cert_api/resources.py
+++ b/certification/cert_api/resources.py
@@ -1083,7 +1083,6 @@ class SubmitRequestResource(Resource):
1083 # submission_id once came from our submission service, we can use a uuid1083 # submission_id once came from our submission service, we can use a uuid
1084 submission_id = uuid()1084 submission_id = uuid()
1085 archive_path = "submission/%s/submission.tar.xz" % submission_id1085 archive_path = "submission/%s/submission.tar.xz" % submission_id
1086 log.error("DEBUG submission_id %s archive_path %s" % (submission_id, archive_path))
1087 # upload submission to swift even before we process them so that we1086 # upload submission to swift even before we process them so that we
1088 # can download failed submissions when trying to help checkbox users1087 # can download failed submissions when trying to help checkbox users
1089 # triage problems1088 # triage problems
@@ -1117,7 +1116,6 @@ class SubmitRequestResource(Resource):
1117 # in case we need to reparse a submission later. (we might want to1116 # in case we need to reparse a submission later. (we might want to
1118 # go the route of legacy submissions and store it in a TextField)1117 # go the route of legacy submissions and store it in a TextField)
1119 json_path = "submission/%s/submission.json" % submission_id1118 json_path = "submission/%s/submission.json" % submission_id
1120 log.error("DEBUG submission_id %s json_path %s" % (submission_id, archive_path))
1121 try:1119 try:
1122 # to be honest I think we should just store this in the db1120 # to be honest I think we should just store this in the db
1123 # but this immitates the go service behavior for now1121 # but this immitates the go service behavior for now

Subscribers

People subscribed via source and target branches

to all changes: