Merge lp:~barryprice/juju-deployer/resolve-OpenStack-1501-option-errors into lp:juju-deployer

Proposed by Barry Price
Status: Merged
Approved by: Barry Price
Approved revision: 220
Merged at revision: 220
Proposed branch: lp:~barryprice/juju-deployer/resolve-OpenStack-1501-option-errors
Merge into: lp:juju-deployer
Prerequisite: lp:~barryprice/juju-deployer/resolve-yaml-warnings
Diff against target: 16 lines (+0/-3)
1 file modified
deployer/tests/test_data/openstack/openstack-1501.yaml (+0/-3)
To merge this branch: bzr merge lp:~barryprice/juju-deployer/resolve-OpenStack-1501-option-errors
Reviewer Review Type Date Requested Status
Benjamin Allot Approve
Canonical IS Reviewers Pending
Review via email: mp+392357@code.launchpad.net

Commit message

Remove no-longer-supported charm options from Openstack bundle to fix tests

To post a comment you must log in.
Revision history for this message
Canonical IS Mergebot (canonical-is-mergebot) wrote :

This merge proposal is being monitored by mergebot. Change the status to Approved to merge.

Revision history for this message
Benjamin Allot (ballot) wrote :

LGTM

review: Approve
Revision history for this message
Canonical IS Mergebot (canonical-is-mergebot) wrote :

Change successfully merged at revision 220

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'deployer/tests/test_data/openstack/openstack-1501.yaml'
2--- deployer/tests/test_data/openstack/openstack-1501.yaml 2016-12-14 08:52:57 +0000
3+++ deployer/tests/test_data/openstack/openstack-1501.yaml 2020-10-16 07:17:20 +0000
4@@ -25,12 +25,9 @@
5 num_units: 1
6 options:
7 osd-devices: /dev/sdb
8- osd-reformat: "yes"
9 ceph-radosgw:
10 charm: cs:trusty/ceph-radosgw
11 num_units: 1
12- options:
13- use-embedded-webserver: true
14 cinder:
15 charm: cs:trusty/cinder
16 num_units: 1

Subscribers

People subscribed via source and target branches