Merge lp://qastaging/~stub/charm-helpers/bug-1214793-service-wrappers into lp://qastaging/charm-helpers
Proposed by
Stuart Bishop
| Status: | Merged | ||||
|---|---|---|---|---|---|
| Merged at revision: | 73 | ||||
| Proposed branch: | lp://qastaging/~stub/charm-helpers/bug-1214793-service-wrappers | ||||
| Merge into: | lp://qastaging/charm-helpers | ||||
| Diff against target: |
140 lines (+64/-13) 2 files modified
charmhelpers/core/host.py (+7/-5) tests/core/test_host.py (+57/-8) |
||||
| To merge this branch: | bzr merge lp://qastaging/~stub/charm-helpers/bug-1214793-service-wrappers | ||||
| Related bugs: |
|
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Jacek Nykis (community) | Approve | ||
|
Review via email:
|
|||
Description of the change
Make wrappers around the service() call, such as service_start() and service_stop(), return the underlying result.
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
Looks fine, all tests check out.