Openstack Mojo Testing

Merge lp:~chris.macnaughton/openstack-mojo-specs/ceph into lp:~ost-maintainers/openstack-mojo-specs/mojo-openstack-specs-1709

  1. ceph
  2. Merge into mojo-openstack-specs-1709
Proposed by Chris MacNaughton
Status: Rejected
Rejected by: Ryan Beisner
Proposed branch: lp:~chris.macnaughton/openstack-mojo-specs/ceph
Merge into: lp:~ost-maintainers/openstack-mojo-specs/mojo-openstack-specs-1709
Diff against target: 90 lines (+7/-7)
7 files modified
specs/storage/ceph/base/icehouse/manifest (+1/-1)
specs/storage/ceph/base/kilo/manifest (+1/-1)
specs/storage/ceph/base/liberty/manifest (+1/-1)
specs/storage/ceph/base/mitaka/manifest (+1/-1)
specs/storage/ceph/base/newton/manifest (+1/-1)
specs/storage/ceph/base/ocata/manifest (+1/-1)
specs/storage/ceph/base/pike/manifest (+1/-1)
To merge this branch: bzr merge lp:~chris.macnaughton/openstack-mojo-specs/ceph
Reviewer Review Type Date Requested Status
Andrew McLeod (community) Approve
OpenStack Charm Testing Maintainers Pending
Review via email: mp+330018@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Andrew McLeod (admcleod) wrote :

I can see that ceph-default.yaml does not exist in this repo, but ceph-base.yaml does, so I'm giving this a +1 - although more info would be useful as to how this name ended up being wrong?

review: Approve
Revision history for this message
Chris MacNaughton (chris.macnaughton) wrote :

I suspect that it was missed when the entire directory structure changed, including the file being renamed from ceph-default.yaml to ceph-base.yaml

Revision history for this message
Chris MacNaughton (chris.macnaughton) wrote :

Specifically in https://bazaar.launchpad.net/~ost-maintainers/openstack-mojo-specs/mojo-openstack-specs-1709/revision/307

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

Thanks for your patch, but this is already resolved in a separate commit.

http://bazaar.launchpad.net/~ost-maintainers/openstack-mojo-specs/mojo-openstack-specs-1709/revision/311

Unmerged revisions

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'specs/storage/ceph/base/icehouse/manifest'
--- specs/storage/ceph/base/icehouse/manifest 2017-02-22 14:27:58 +0000
+++ specs/storage/ceph/base/icehouse/manifest 2017-08-31 15:16:44 +0000
@@ -2,7 +2,7 @@
2collect config=collect-ceph-default2collect config=collect-ceph-default
33
4# Use juju deployer with ceph-default.yaml bundle4# Use juju deployer with ceph-default.yaml bundle
5deploy config=ceph-default.yaml delay=0 wait=False target=${MOJO_SERIES}-icehouse5deploy config=ceph-base.yaml delay=0 wait=False target=${MOJO_SERIES}-icehouse
66
7# Check juju statuses are green and that hooks have finished7# Check juju statuses are green and that hooks have finished
8verify config=check_juju.py8verify config=check_juju.py
99
=== modified file 'specs/storage/ceph/base/kilo/manifest'
--- specs/storage/ceph/base/kilo/manifest 2017-02-22 14:27:58 +0000
+++ specs/storage/ceph/base/kilo/manifest 2017-08-31 15:16:44 +0000
@@ -2,7 +2,7 @@
2collect config=collect-ceph-default2collect config=collect-ceph-default
33
4# Use juju deployer with ceph-default.yaml bundle4# Use juju deployer with ceph-default.yaml bundle
5deploy config=ceph-default.yaml delay=0 wait=False target=${MOJO_SERIES}-kilo5deploy config=ceph-base.yaml delay=0 wait=False target=${MOJO_SERIES}-kilo
66
7# Check juju statuses are green and that hooks have finished7# Check juju statuses are green and that hooks have finished
8verify config=check_juju.py8verify config=check_juju.py
99
=== modified file 'specs/storage/ceph/base/liberty/manifest'
--- specs/storage/ceph/base/liberty/manifest 2017-02-22 14:27:58 +0000
+++ specs/storage/ceph/base/liberty/manifest 2017-08-31 15:16:44 +0000
@@ -2,7 +2,7 @@
2collect config=collect-ceph-default2collect config=collect-ceph-default
33
4# Use juju deployer with ceph-default.yaml bundle4# Use juju deployer with ceph-default.yaml bundle
5deploy config=ceph-default.yaml delay=0 wait=False target=${MOJO_SERIES}-liberty5deploy config=ceph-base.yaml delay=0 wait=False target=${MOJO_SERIES}-liberty
66
7# Check juju statuses are green and that hooks have finished7# Check juju statuses are green and that hooks have finished
8verify config=check_juju.py8verify config=check_juju.py
99
=== modified file 'specs/storage/ceph/base/mitaka/manifest'
--- specs/storage/ceph/base/mitaka/manifest 2017-02-22 14:27:58 +0000
+++ specs/storage/ceph/base/mitaka/manifest 2017-08-31 15:16:44 +0000
@@ -2,7 +2,7 @@
2collect config=collect-ceph-default2collect config=collect-ceph-default
33
4# Use juju deployer with ceph-default.yaml bundle4# Use juju deployer with ceph-default.yaml bundle
5deploy config=ceph-default.yaml delay=0 wait=False target=${MOJO_SERIES}-mitaka5deploy config=ceph-base.yaml delay=0 wait=False target=${MOJO_SERIES}-mitaka
66
7# Check juju statuses are green and that hooks have finished7# Check juju statuses are green and that hooks have finished
8verify config=check_juju.py8verify config=check_juju.py
99
=== modified file 'specs/storage/ceph/base/newton/manifest'
--- specs/storage/ceph/base/newton/manifest 2017-02-22 14:27:58 +0000
+++ specs/storage/ceph/base/newton/manifest 2017-08-31 15:16:44 +0000
@@ -2,7 +2,7 @@
2collect config=collect-ceph-default2collect config=collect-ceph-default
33
4# Use juju deployer with ceph-default.yaml bundle4# Use juju deployer with ceph-default.yaml bundle
5deploy config=ceph-default.yaml delay=0 wait=False target=${MOJO_SERIES}-newton5deploy config=ceph-base.yaml delay=0 wait=False target=${MOJO_SERIES}-newton
66
7# Check juju statuses are green and that hooks have finished7# Check juju statuses are green and that hooks have finished
8verify config=check_juju.py8verify config=check_juju.py
99
=== modified file 'specs/storage/ceph/base/ocata/manifest'
--- specs/storage/ceph/base/ocata/manifest 2017-08-30 17:25:03 +0000
+++ specs/storage/ceph/base/ocata/manifest 2017-08-31 15:16:44 +0000
@@ -2,7 +2,7 @@
2collect config=collect-ceph-default2collect config=collect-ceph-default
33
4# Use juju deployer with ceph-default.yaml bundle4# Use juju deployer with ceph-default.yaml bundle
5deploy config=ceph-default.yaml delay=0 wait=False target=${MOJO_SERIES}-ocata5deploy config=ceph-base.yaml delay=0 wait=False target=${MOJO_SERIES}-ocata
66
7# Check juju statuses are green and that hooks have finished7# Check juju statuses are green and that hooks have finished
8verify config=check_juju.py8verify config=check_juju.py
99
=== modified file 'specs/storage/ceph/base/pike/manifest'
--- specs/storage/ceph/base/pike/manifest 2017-08-30 17:25:03 +0000
+++ specs/storage/ceph/base/pike/manifest 2017-08-31 15:16:44 +0000
@@ -2,7 +2,7 @@
2collect config=collect-ceph-default2collect config=collect-ceph-default
33
4# Use juju deployer with ceph-default.yaml bundle4# Use juju deployer with ceph-default.yaml bundle
5deploy config=ceph-default.yaml delay=0 wait=False target=${MOJO_SERIES}-pike5deploy config=ceph-base.yaml delay=0 wait=False target=${MOJO_SERIES}-pike
66
7# Check juju statuses are green and that hooks have finished7# Check juju statuses are green and that hooks have finished
8verify config=check_juju.py8verify config=check_juju.py

Subscribers

People subscribed via source and target branches