Merge lp:~james-w/tarmac/lock-contention into lp:tarmac

Proposed by James Westby
Status: Merged
Approved by: Paul Hummer
Approved revision: 404
Merged at revision: 405
Proposed branch: lp:~james-w/tarmac/lock-contention
Merge into: lp:tarmac
Prerequisite: lp:~james-w/tarmac/one-at-a-time
Diff against target: 21 lines (+3/-1)
1 file modified
tarmac/bin/commands.py (+3/-1)
To merge this branch: bzr merge lp:~james-w/tarmac/lock-contention
Reviewer Review Type Date Requested Status
Paul Hummer Approve
Review via email: mp+146181@code.launchpad.net

This proposal supersedes a proposal from 2013-01-31.

Commit message

Ignore LockContention errors when trying to merge a branch.

Description of the change

Ignore LockContention errors when trying to merge a branch.

(from the u1 fork)

To post a comment you must log in.
Revision history for this message
Paul Hummer (rockstar) :
review: Approve
Revision history for this message
Paul Hummer (rockstar) wrote :

The prerequisite lp:~james-w/tarmac/one-at-a-time has not yet been merged into lp:tarmac.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'tarmac/bin/commands.py'
--- tarmac/bin/commands.py 2013-02-01 17:28:25 +0000
+++ tarmac/bin/commands.py 2013-02-01 17:28:25 +0000
@@ -5,7 +5,7 @@
5import re5import re
66
7from bzrlib.commands import Command7from bzrlib.commands import Command
8from bzrlib.errors import PointlessMerge8from bzrlib.errors import PointlessMerge, LockContention
9from bzrlib.help import help_commands9from bzrlib.help import help_commands
10from launchpadlib.launchpad import Launchpad10from launchpadlib.launchpad import Launchpad
11from launchpadlib.uris import (LPNET_SERVICE_ROOT,11from launchpadlib.uris import (LPNET_SERVICE_ROOT,
@@ -374,6 +374,8 @@
374 # If we've been asked to only merge one branch, then exit.374 # If we've been asked to only merge one branch, then exit.
375 if merged and self.config.one:375 if merged and self.config.one:
376 break376 break
377 except LockContention:
378 continue
377 except Exception, error:379 except Exception, error:
378 self.logger.error(380 self.logger.error(
379 'An error occurred trying to merge %s: %s',381 'An error occurred trying to merge %s: %s',

Subscribers

People subscribed via source and target branches