[Airship-discuss] Divingbell overrides functionality regression - Assistance needed

Roman Gorshunov paye600 at gmail.com
Thu Mar 7 13:17:20 UTC 2019


Playing a little bit, I have just found out that using helm version
v2.13.0-rc.1 and higher, AND `mergeOverwrite` allows to pass linting.
Log: http://paste.openstack.org/show/747402/

I've submitted a test PS to see if it would pass standard voting
gates: https://review.openstack.org/#/c/641656/
The non-voting `openstack-helm-infra-airship-divingbell` gate should
still fail, as it would use old `merge`.

Waiting for gates to complete the work.

-- Roman Gorshunov

On Thu, Mar 7, 2019 at 1:55 PM Roman Gorshunov <paye600 at gmail.com> wrote:
>
> > Just to confirm, were you using Helm 1.12.x when you tried replacing `merge` with `mergOverwrite`?
> I used helm 2.12.3.I confirm that mergeOverwrite does not work.
>
> Link with test: http://paste.openstack.org/show/747400/
> Details:
> - line  4: helm client version v2.12.3 (this version is used in
> openstack-helm-infra by default)
> - line 16: latest openstack-helm-infra (commit id
> e836707ad070e910d6130eced066bc0e50481d2b)
> - line 30: `helm lint` of helm-toolkit
> - line 33: ./templates/utils/_daemonset_overrides.tpl uses `merge`
> - line 40: replace `merge` with `mergeOverwrite`
> - line 41: confirm replacement
> - line 50: linting fails with error `function "mergeOverwrite" not defined`
>
> Thank you for your help, Craig.
>
> -- Roman Gorshunov



More information about the Airship-discuss mailing list