Merge lp:~chris.macnaughton/openstack-mojo-specs/update-ceph-test into lp:openstack-mojo-specs
Proposed by
Chris MacNaughton
Status: | Merged |
---|---|
Merged at revision: | 311 |
Proposed branch: | lp:~chris.macnaughton/openstack-mojo-specs/update-ceph-test |
Merge into: | lp:openstack-mojo-specs |
Diff against target: |
77 lines (+16/-7) 3 files modified
helper/bundles/ceph-charm-migration.yaml (+11/-1) helper/tests/test_ceph_store.py (+2/-6) specs/storage/ceph/charm_migration/pike/manifest (+3/-0) |
To merge this branch: | bzr merge lp:~chris.macnaughton/openstack-mojo-specs/update-ceph-test |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Ryan Beisner | Approve | ||
Review via email: mp+332421@code.launchpad.net |
Description of the change
The bundles are updated to deploy the architecture we intend, with monitors going into containers on the existing machines, and ceph-osd going onto the metal on those machines. This change also adds a check to ensure the juju deployment is healthy before finishing tests
To post a comment you must log in.
Approved, merging. Fixing two trivial items in-flight:
* One bit of lint. Please be sure to run `make lint` with spec mods. tests/test_ ceph_store. py:8:1: E302 expected 2 blank lines, found 1
helper/
* Removed the commented placement lines from the ceph migration bundle.
http:// 10.245. 162.49: 8080/view/ MojoMatrix/ job/mojo_ runner/ 1816/