Merge lp:~mwhudson/launchpad/code-import-workers-copy-too-much into lp:launchpad
Proposed by
Michael Hudson-Doyle
Status: | Merged |
---|---|
Approved by: | Tim Penhey |
Approved revision: | no longer in the source branch. |
Merged at revision: | not available |
Proposed branch: | lp:~mwhudson/launchpad/code-import-workers-copy-too-much |
Merge into: | lp:launchpad |
Diff against target: |
59 lines (+23/-3) 3 files modified
lib/lp/code/model/tests/test_diff.py (+3/-1) lib/lp/codehosting/codeimport/tests/test_worker.py (+15/-0) lib/lp/codehosting/codeimport/worker.py (+5/-2) |
To merge this branch: | bzr merge lp:~mwhudson/launchpad/code-import-workers-copy-too-much |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Tim Penhey (community) | Approve | ||
Review via email: mp+24282@code.launchpad.net |
Commit message
Only copy the .bzr directory of an existing import, avoiding any backup.bzr directories that may be present
Description of the change
Hi,
The rollout of a cherry pick to the code import system last week broke all CSCVS imports that had been upgraded on escudero because the .backup.bzr and/or backup.bzr directories were copied to the slaves along with the .bzr directory, which then broke the import when it tried to commit.
As a band-aid I got mbarnett to remove all .bzr.backup and backup.bzr directories on escudero, but luckily a real fix is easy too -- see the diff!
Cheers,
mwh
To post a comment you must log in.
The fix is wrong, rather than using root_transport, if you want just
the control dir, use the 'transport' attribute instead.