Merge lp://staging/~niedbalski/python-jujuclient/lp-1456332 into lp://staging/python-jujuclient
Proposed by
Jorge Niedbalski
Status: | Superseded | ||||
---|---|---|---|---|---|
Proposed branch: | lp://staging/~niedbalski/python-jujuclient/lp-1456332 | ||||
Merge into: | lp://staging/python-jujuclient | ||||
Diff against target: |
137 lines (+94/-5) 2 files modified
jujuclient.py (+30/-5) test_jujuclient.py (+64/-0) |
||||
To merge this branch: | bzr merge lp://staging/~niedbalski/python-jujuclient/lp-1456332 | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Felipe Reyes (community) | Needs Fixing | ||
Tim Van Steenburgh (community) | Needs Fixing | ||
Review via email:
|
This proposal has been superseded by a proposal from 2015-05-18.
Description of the change
This patch fixes LP: #1456332
To post a comment you must log in.
One minor quibble: it's not clear what happens if you call Run without providing any machines, services, or units. Is that an error from Juju, or is it the equivalent of calling RunOnAllMachines? It would be great to clarify this in the run() docstring.
Otherwise looks good, pending a successful test run.