Merge ~hemanth-n/stsstack-bundles:1888100 into stsstack-bundles:master

Proposed by Hemanth Nakkina
Status: Merged
Merged at revision: d4c07d5741a636d2267ac085fcbb6e961210fd46
Proposed branch: ~hemanth-n/stsstack-bundles:1888100
Merge into: stsstack-bundles:master
Diff against target: 34 lines (+10/-2)
2 files modified
common/helpers (+9/-1)
openstack/pipeline/02configure (+1/-1)
Reviewer Review Type Date Requested Status
Edward Hope-Morley Approve
Review via email: mp+387634@code.launchpad.net

Commit message

Update parameter __OCTAVIA_RETROFIT_UCA__ even when release is not specified

To post a comment you must log in.
Revision history for this message
Edward Hope-Morley (hopem) wrote :

lgtm

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/common/helpers b/common/helpers
2index 931b423..e71c27c 100644
3--- a/common/helpers
4+++ b/common/helpers
5@@ -347,7 +347,15 @@ get_uca_release ()
6 # lts s+r means no uca
7 ltsmatch "$s" "$r" && return
8 # else its uca
9- echo $release
10+ echo $r
11+}
12+
13+get_ost_release ()
14+{
15+ local r=`get_release`
16+ local s=`get_series`
17+ [ -n "$r" ] || r=${lts[$s]:-${nonlts[$s]:-""}}
18+ echo "$r"
19 }
20
21 cache ()
22diff --git a/openstack/pipeline/02configure b/openstack/pipeline/02configure
23index e41319b..0ddbe2d 100644
24--- a/openstack/pipeline/02configure
25+++ b/openstack/pipeline/02configure
26@@ -301,7 +301,7 @@ do
27 if ! has_opt --no-octavia-diskimage-retrofit; then
28 # By default we let retrofit use images uploaded by the post-deploy configure script.
29 MOD_OVERLAYS+=( "octavia-diskimage-retrofit.yaml" )
30- MOD_PARAMS[__OCTAVIA_RETROFIT_UCA__]=`get_uca_release`
31+ MOD_PARAMS[__OCTAVIA_RETROFIT_UCA__]=`get_ost_release`
32 if ! has_opt --octavia-diskimage-retrofit-glance-simplestreams; then
33 MOD_OVERLAYS+=( "octavia-diskimage-retrofit-glance.yaml" )
34 fi

Subscribers

People subscribed via source and target branches