juju-ci-tools

Merge lp://staging/~sinzui/juju-ci-tools/update-when-slave-down into lp://staging/juju-ci-tools

  1. update-when-slave-down
  2. Merge into trunk
Proposed by Curtis Hovey
Status: Merged
Merged at revision: 656
Proposed branch: lp://staging/~sinzui/juju-ci-tools/update-when-slave-down
Merge into: lp://staging/juju-ci-tools
Diff against target: 35 lines (+12/-1)
2 files modified
deploy_stack.py (+1/-1)
update-jenkins-branches.bash (+11/-0)
To merge this branch: bzr merge lp://staging/~sinzui/juju-ci-tools/update-when-slave-down
Reviewer Review Type Date Requested Status
Aaron Bentley (community) Approve
Review via email: mp+234491@code.staging.launchpad.net

Description of the change

Change update-jenkins-branches.bash to try to update all hosts, then report which failed. This is the example put from my run of this when the arm64 machine was down.

These hosts were skipped because thee was an error
 10.229.41.215

To post a comment you must log in.
Revision history for this message
Aaron Bentley (abentley) wrote :

Works for me. One style suggestion below.

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