Merge lp://staging/~sinzui/juju-ci-tools/reliable-weekly into lp://staging/juju-ci-tools

Proposed by Curtis Hovey
Status: Merged
Merged at revision: 1011
Proposed branch: lp://staging/~sinzui/juju-ci-tools/reliable-weekly
Merge into: lp://staging/juju-ci-tools
Diff against target: 180 lines (+55/-18)
4 files modified
candidate.py (+14/-3)
setup-slave.bash (+1/-1)
test_candidate.py (+39/-13)
test_joyent.py (+1/-1)
To merge this branch: bzr merge lp://staging/~sinzui/juju-ci-tools/reliable-weekly
Reviewer Review Type Date Requested Status
John George (community) Approve
Review via email: mp+263598@code.staging.launchpad.net

Description of the change

Retry weekly stream publication.

The branch extracts the call to publish-streams.bash and tries it up to 3 times. This will make weekly publication as reliable as testing publication.

I added patch('sys.stdout') to several tests that were printing in the test output.

PS, there is a drive by fix for the setup-slave script. I encountered this issue setting of the certification-slave.

To post a comment you must log in.
1009. By Curtis Hovey

Don't create command more than once.

Revision history for this message
John George (jog) wrote :

approve.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches