Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(469)

Unified Diff: pending_manager.py

Issue 130983010: Reduce spam from CQ. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/tools/commit-queue
Patch Set: Addressing the comments Created 6 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | tests/pending_manager_test.py » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: pending_manager.py
diff --git a/pending_manager.py b/pending_manager.py
index 6f61d4b8641745cd1b41dce570bed1dcc5824ba9..def87d80f4690aeff7e25b23182cc28850106707 100644
--- a/pending_manager.py
+++ b/pending_manager.py
@@ -544,18 +544,11 @@ class PendingManager(object):
self.context.rietveld.close_issue(pending.issue)
self.context.rietveld.update_description(
pending.issue, issue_desc.description)
- self.context.rietveld.add_comment(
- pending.issue, 'Change committed as %s' % pending.revision)
except (urllib2.HTTPError, urllib2.URLError) as e:
# Ignore AppEngine flakiness.
logging.warning('Unable to fully close the issue')
- # And finally remove the issue. If the close_issue() call above failed,
- # it is possible the dashboard will be confused but it is harmless.
- try:
- self.queue.get(pending.issue)
- except KeyError:
- logging.error('Internal inconsistency for %d', pending.issue)
- self.queue.remove(pending.issue)
+ self._discard_pending(pending,
+ 'Change committed as %s' % pending.revision)
except (
checkout.PatchApplicationFailed, patch.UnsupportedPatchFormat) as e:
raise base.DiscardPending(pending, str(e))
« no previous file with comments | « no previous file | tests/pending_manager_test.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698