Merge ~hopem/stsstack-bundles:bug/1896808 into stsstack-bundles:master

Proposed by Edward Hope-Morley
Status: Merged
Merged at revision: 2da1abf99829f9096d0595e7e8f596822d85c09b
Proposed branch: ~hopem/stsstack-bundles:bug/1896808
Merge into: stsstack-bundles:master
Diff against target: 38 lines (+3/-2)
3 files modified
openstack/module_defaults (+1/-0)
openstack/pipeline/02configure (+1/-1)
overlays/mysql-innodb-cluster.yaml (+1/-1)
Reviewer Review Type Date Requested Status
Ponnuvel Palaniyappan (community) Approve
Jolly Bundlers Pending
Review via email: mp+391278@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Ponnuvel Palaniyappan (pponnuvel) wrote :

Tested it and it works now. Thanks.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
diff --git a/openstack/module_defaults b/openstack/module_defaults
index 79f0cf7..c7ece87 100644
--- a/openstack/module_defaults
+++ b/openstack/module_defaults
@@ -20,6 +20,7 @@ MOD_PARAMS[__NUM_HEAT_UNITS__]=1
20MOD_PARAMS[__NUM_HORIZON_UNITS__]=120MOD_PARAMS[__NUM_HORIZON_UNITS__]=1
21MOD_PARAMS[__NUM_KEYSTONE_UNITS__]=121MOD_PARAMS[__NUM_KEYSTONE_UNITS__]=1
22MOD_PARAMS[__NUM_MYSQL_UNITS__]=122MOD_PARAMS[__NUM_MYSQL_UNITS__]=1
23MOD_PARAMS[__NUM_MYSQL_INNODB_UNITS__]=3 # absolute min required
23MOD_PARAMS[__NUM_NEUTRON_API_UNITS__]=124MOD_PARAMS[__NUM_NEUTRON_API_UNITS__]=1
24MOD_PARAMS[__NUM_NOVACC_UNITS__]=125MOD_PARAMS[__NUM_NOVACC_UNITS__]=1
25MOD_PARAMS[__NUM_RABBIT_UNITS__]=126MOD_PARAMS[__NUM_RABBIT_UNITS__]=1
diff --git a/openstack/pipeline/02configure b/openstack/pipeline/02configure
index 96a3ab0..1faaf36 100644
--- a/openstack/pipeline/02configure
+++ b/openstack/pipeline/02configure
@@ -254,7 +254,7 @@ do
254 conflicts_with $1 --mysql-ha*254 conflicts_with $1 --mysql-ha*
255 assert_min_release train mysql-innodb-cluster255 assert_min_release train mysql-innodb-cluster
256 # NOTE: 3 is actually the absolute minimum units required256 # NOTE: 3 is actually the absolute minimum units required
257 get_units $1 __NUM_MYSQL_UNITS__ 3257 get_units $1 __NUM_MYSQL_INNODB_UNITS__ 3
258 MOD_OVERLAYS+=( "mysql-innodb-cluster.yaml")258 MOD_OVERLAYS+=( "mysql-innodb-cluster.yaml")
259 ;;259 ;;
260 --no-ml2dns) #__OPT__260 --no-ml2dns) #__OPT__
diff --git a/overlays/mysql-innodb-cluster.yaml b/overlays/mysql-innodb-cluster.yaml
index fd75c92..8fb498b 100644
--- a/overlays/mysql-innodb-cluster.yaml
+++ b/overlays/mysql-innodb-cluster.yaml
@@ -5,7 +5,7 @@ applications:
5 mysql: # we call it 'mysql' since that's how other applications address this one.5 mysql: # we call it 'mysql' since that's how other applications address this one.
6 charm: cs:~openstack-charmers-next/mysql-innodb-cluster6 charm: cs:~openstack-charmers-next/mysql-innodb-cluster
7 constraints: mem=4G7 constraints: mem=4G
8 num_units: __NUM_MYSQL_UNITS__8 num_units: __NUM_MYSQL_INNODB_UNITS__
9 options:9 options:
10 source: *source10 source: *source
11 innodb-buffer-pool-size: 50%11 innodb-buffer-pool-size: 50%

Subscribers

People subscribed via source and target branches