Merge lp:~corey.bryant/charms/trusty/cinder/remove-missing into lp:~openstack-charmers-archive/charms/trusty/cinder/next
Proposed by
Corey Bryant
Status: | Merged | ||||
---|---|---|---|---|---|
Merged at revision: | 56 | ||||
Proposed branch: | lp:~corey.bryant/charms/trusty/cinder/remove-missing | ||||
Merge into: | lp:~openstack-charmers-archive/charms/trusty/cinder/next | ||||
Diff against target: |
251 lines (+62/-16) 5 files modified
config.yaml (+12/-2) hooks/cinder_hooks.py (+2/-1) hooks/cinder_utils.py (+19/-2) unit_tests/test_cinder_hooks.py (+3/-2) unit_tests/test_cinder_utils.py (+26/-9) |
||||
To merge this branch: | bzr merge lp:~corey.bryant/charms/trusty/cinder/remove-missing | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
James Page | Needs Fixing | ||
OpenStack Charmers | Pending | ||
Review via email: mp+237439@code.launchpad.net |
Description of the change
OIL has hit failures where physical volumes (from a previous deployment) are missing from the volume group on the current deployment, and vgextend fails. This patch fixes that by adding an option that requests that missing physical volumes be removed from the volume group.
To post a comment you must log in.
Marking WIP - set back to 'Needs review' when ready for re-review.