Tarmac should merge into a fresh branch

Bug #488297 reported by Francis J. Lacoste
10
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Tarmac
Fix Released
High
Paul Hummer

Bug Description

When Tarmac reported conflicts when it merged my branch. My branch has trunk already merged in so is really conflict-free.
The conflicts are due because of built files in the branch. IOW, to ensure a clean merge, Tarmac should start from a fresh branch/checkout.

This is the error report I got:

Attempt to merge lp:~flacoste/oops-tools/buildoutification into lp:~launchpad-pqm/oops-tools/trunk failed due to merge conflicts:

deleting parent in lib
deleting parent in lib/zope
deleting parent in lib/zope/deprecation
deleting parent in lib/zope/i18nmessageid
deleting parent in lib/zope/interface
deleting parent in lib/zope/schema
deleting parent in lib/zope/testbrowser
deleting parent in lib/zope/testing

Related branches

Revision history for this message
Paul Hummer (rockstar) wrote :

Did Tarmac explode beforehand? If so, the way it failed apparently never got to cleaning up the trunk, breaking the next commit.

It CAN use a fresh checkout, or it can be cached. I'll have to look at the codepaths again and make sure all failures still clean up the trunk.

Changed in tarmac:
status: New → Triaged
importance: Undecided → High
milestone: none → tarmac-0.3
Revision history for this message
Francis J. Lacoste (flacoste) wrote :

I don't know, it was the first failure I got and Matsubara did land successfully some changes earlier. There might have been failures in between, but I wouldn't know. If there is a log file that we should look into, I can ask a losa to peek.

Revision history for this message
Francis J. Lacoste (flacoste) wrote :

Ok, I can confirm that cleanup doesn't happen after an error. After each error I get, we needed to run make clean in the chroot so that the merge can complete successfully on the next attempt.

Revision history for this message
Paul Hummer (rockstar) wrote :

Fixed in revno 264

Changed in tarmac:
status: Triaged → Fix Committed
assignee: nobody → Paul Hummer (rockstar)
Paul Hummer (rockstar)
Changed in tarmac:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.