openstack-helm/helm-toolkit
Artur Korzeniewski 7e532bdb11 Helm-toolkit: Fixed upgrade daemonset snippet
The issue was with type of upgrade strategy, which was rendered as empty.
It was due to wrong access to the pod_replacement_strategy, which was
out of modified scope:

      daemonsets:
        pod_replacement_strategy: RollingUpdate
        dhcp_agent:
          enabled: false
          min_ready_seconds: 0
          max_unavailable: 1

and the snippet was modifying the scope to daemonset.dhcp_agent, then
trying to access daemonsets.dhcpagent.pod_replacement_strategy, which was
not there:

{{- $upgradeMap := index $envAll.Values.pod.lifecycle.upgrades.daemonsets $component }}
updateStrategy:
  type: {{ .pod_replacement_strategy }}

The fix is to evaluate the common variable before entering the scope
modification.

{{- $pod_replacement_strategy := $envAll.Values.pod.lifecycle.upgrades.daemonsets.pod_replacement_strategy -}}
type: {{ $pod_replacement_strategy }}

Change-Id: I78937b1bf1e0d50ca372de095582c71105d76331
Closes-Bug: 1705662
2017-07-21 11:09:36 +02:00
..
templates Helm-toolkit: Fixed upgrade daemonset snippet 2017-07-21 11:09:36 +02:00
.gitignore Rename common chart, and update all references to functions within it. (#167) 2017-02-14 16:52:38 -08:00
.helmignore Rename common chart, and update all references to functions within it. (#167) 2017-02-14 16:52:38 -08:00
Chart.yaml Remove executable flag from Chart.yaml 2017-06-06 11:43:25 -05:00
Makefile add license headers to helm-toolkit (#217) 2017-02-20 07:06:34 -08:00
requirements.yaml add license headers to helm-toolkit (#217) 2017-02-20 07:06:34 -08:00
values.yaml add license headers to helm-toolkit (#217) 2017-02-20 07:06:34 -08:00