When setting --ssl and specifying unit count for a ha unit, the number of units defaults to 1

Bug #1882562 reported by Arif Ali
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
stsstack-bundles
Fix Released
High
Edward Hope-Morley

Bug Description

When specifying --ssl, then the number of units specified by a specific ha overlay is overwritten due to the line 338 in 01configure script and also adds --ha to the options. This in turn adds the cinder-ha, glance-ha, keystone-ha, neutron-api-ha, nova-cloud-controller-ha with one unit each as well as defaulting to 1 for keystone-ha.

This was unexpected, as I only wanted to test ssl with keystone-ha

Here was my script

```
./generate-bundle.sh \
  -n ldap \
  -s bionic \
  --use-stable-charms \
  --overlay-network-type vxlan \
  --create-model \
  --num-compute 3 \
  --openstack-dashboard \
  --keystone-ha:3 \
  --ssl \
  --ldap
```

Below is my debugging to see the the units, and what was going on in the get_units function in helpers file

```
ubuntu@arif-ali-bastion:~/stsstack-bundles/openstack$ ./generate_ldap.sh
+ rm -rf ./b/ldap
+ ./generate-bundle.sh -n ldap -s bionic --use-stable-charms --overlay-network-type vxlan --create-model --num-compute 3 --ssl --openstack-dashboard --keystone-ha:3 --ldap
02configure ..... Using existing ssl certificates in ssl/openstack-ldap
--keystone-ha:3 __NUM_KEYSTONE_UNITS__ 3
--keystone-ha:3 __NUM_KEYSTONE_UNITS__ 3
REQUIRED: address of ldap server (leave blank to set later):
--ha:1 __NUM_HA_UNITS__ 3
--ha:1 __NUM_HA_UNITS__ 3
--cinder-ha:1 __NUM_CINDER_UNITS__ 3
--cinder-ha:1 __NUM_CINDER_UNITS__ 3
--glance-ha:1 __NUM_GLANCE_UNITS__ 3
--glance-ha:1 __NUM_GLANCE_UNITS__ 3
--keystone-ha:1 __NUM_KEYSTONE_UNITS__ 3
--keystone-ha:1 __NUM_KEYSTONE_UNITS__ 3
--neutron-api-ha:1 __NUM_NEUTRON_API_UNITS__ 3
--neutron-api-ha:1 __NUM_NEUTRON_API_UNITS__ 3
--nova-cloud-controller-ha:1 __NUM_NOVACC_UNITS__ 3
--nova-cloud-controller-ha:1 __NUM_NOVACC_UNITS__ 3
--openstack-dashboard-ha:1 __NUM_HORIZON_UNITS__ 3
--openstack-dashboard-ha:1 __NUM_HORIZON_UNITS__ 3
Juju model 'ldap' already exists and is the current context - skipping create

Created bionic-queens bundle and overlays (using stable charms):
  + neutron-gateway.yaml
  + neutron-openvswitch.yaml
  + mysql.yaml
  + openstack-dashboard.yaml
  + keystone-ha.yaml
  + ldap.yaml
  + ldap-test-fixture.yaml
  + neutron-ml2dns.yaml
  + neutron-ml2dns-gateway.yaml
  + neutron-ml2dns-openvswitch.yaml
  + cinder-ha.yaml
  + glance-ha.yaml
  + neutron-api-ha.yaml
  + nova-cloud-controller-ha.yaml
  + memcached.yaml
  + openstack-dashboard-ha.yaml

Command to deploy:
juju deploy ./b/ldap/openstack.yaml --overlay ./b/ldap/o/neutron-gateway.yaml --overlay ./b/ldap/o/neutron-openvswitch.yaml --overlay ./b/ldap/o/mysql.yaml --overlay ./b/ldap/o/openstack-dashboard.yaml --overlay
 ./b/ldap/o/keystone-ha.yaml --overlay ./b/ldap/o/ldap.yaml --overlay ./b/ldap/o/ldap-test-fixture.yaml --overlay ./b/ldap/o/neutron-ml2dns.yaml --overlay ./b/ldap/o/neutron-ml2dns-gateway.yaml --overlay ./b/lda
p/o/neutron-ml2dns-openvswitch.yaml --overlay ./b/ldap/o/cinder-ha.yaml --overlay ./b/ldap/o/glance-ha.yaml --overlay ./b/ldap/o/neutron-api-ha.yaml --overlay ./b/ldap/o/nova-cloud-controller-ha.yaml --overlay .
/b/ldap/o/memcached.yaml --overlay ./b/ldap/o/openstack-dashboard-ha.yaml --channel=stable
```

Related branches

Changed in stsstack-bundles:
assignee: nobody → Edward Hope-Morley (hopem)
status: New → In Progress
importance: Undecided → Critical
importance: Critical → High
Changed in stsstack-bundles:
status: In Progress → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.