Merge lp://staging/~sinzui/juju-ci-tools/fix-win into lp://staging/juju-ci-tools
Proposed by
Curtis Hovey
| Status: | Merged |
|---|---|
| Merged at revision: | 997 |
| Proposed branch: | lp://staging/~sinzui/juju-ci-tools/fix-win |
| Merge into: | lp://staging/juju-ci-tools |
| Diff against target: |
32 lines (+5/-4) 1 file modified
test-win-client (+5/-4) |
| To merge this branch: | bzr merge lp://staging/~sinzui/juju-ci-tools/fix-win |
| Related bugs: |
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| John George (community) | Approve | ||
|
Review via email:
|
|||
Description of the change
Fix test-win-client exit code, script path, and deps.
The convoluted rules to retry when ssh fails broke the test to bootstrap with a win client.
* The exit code seen clearly to be an error in the console is not the same same exit code jenkins sees.
* The script to run the test is not run because the win path is corrupted because the \\ is interpreteted twice.
* The additions of get_ami and remote were not added as deploy-stack requires.
To post a comment you must log in.
Preview Diff
[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
approved.