Merge lp://qastaging/~gz/pyjuju/os_scheduler_hints_json_fix into lp://qastaging/pyjuju
Proposed by
Martin Packman
| Status: | Merged |
|---|---|
| Approved by: | Kapil Thangavelu |
| Approved revision: | 617 |
| Merged at revision: | 617 |
| Proposed branch: | lp://qastaging/~gz/pyjuju/os_scheduler_hints_json_fix |
| Merge into: | lp://qastaging/pyjuju |
| Diff against target: |
37 lines (+6/-4) 2 files modified
juju/providers/openstack/client.py (+3/-2) juju/providers/openstack/tests/test_client.py (+3/-2) |
| To merge this branch: | bzr merge lp://qastaging/~gz/pyjuju/os_scheduler_hints_json_fix |
| Related bugs: |
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Kapil Thangavelu (community) | Approve | ||
| John A Meinel | Approve | ||
|
Review via email:
|
|||
Description of the change
Fix format of the scheduler hints passed in the json dict when creating a server.
This is the juju part of getting hints working, see the mailing list post:
<https:/
There is some discussion about designing a neater abstraction, this is just fixing the existing code in the interim.
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
Matches my understanding of how this should work.