Merge lp://staging/~chris.macnaughton/openstack-mojo-specs/update-ceph-test into lp://staging/openstack-mojo-specs

Proposed by Chris MacNaughton
Status: Merged
Merged at revision: 311
Proposed branch: lp://staging/~chris.macnaughton/openstack-mojo-specs/update-ceph-test
Merge into: lp://staging/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://staging/~chris.macnaughton/openstack-mojo-specs/update-ceph-test
Reviewer Review Type Date Requested Status
Ryan Beisner Approve
Review via email: mp+332421@code.staging.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.
309. By Chris MacNaughton

clean up specs, update bundle, fail when ceph charm removal fails

310. By Chris MacNaughton

do not use lxd placement yet as it seems broken still

Revision history for this message
Ryan Beisner (1chb1n) wrote :

Approved, merging. Fixing two trivial items in-flight:

* One bit of lint. Please be sure to run `make lint` with spec mods.
  helper/tests/test_ceph_store.py:8:1: E302 expected 2 blank lines, found 1

* Removed the commented placement lines from the ceph migration bundle.

http://10.245.162.49:8080/view/MojoMatrix/job/mojo_runner/1816/

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches