Merge lp://staging/~abentley/juju-ci-tools/stop-copying into lp://staging/juju-ci-tools
Proposed by
Aaron Bentley
| Status: | Merged |
|---|---|
| Merged at revision: | 1843 |
| Proposed branch: | lp://staging/~abentley/juju-ci-tools/stop-copying |
| Merge into: | lp://staging/juju-ci-tools |
| Diff against target: |
34 lines (+5/-9) 1 file modified
assess_win_client.py (+5/-9) |
| To merge this branch: | bzr merge lp://staging/~abentley/juju-ci-tools/stop-copying |
| Related bugs: |
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Curtis Hovey (community) | code | Approve | |
|
Review via email:
|
|||
Commit message
Use the existing juju-ci-tools rather than copying some of it.
Description of the change
This branch stops assess_win_client copying juju-ci-tools.
Instead, it uses the copy of juju-ci-tools already in place on the destination.
This means that assess_win_client no longer has specific knowledge of the dependencies of deploy_job and their locations, so that these can be changed freely.
split-jujupy moves files around, so either
a) assess_win_client needs to stop knowing where files are.
b) assess_win_client needs to be updated with the new locations.
I think a) is better in the long run.
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
Thank you.