banzaicloud / banzai-charts

Curated list of Banzai Cloud Helm charts used by the Pipeline Platform
Apache License 2.0
368 stars 283 forks source link

Fixed Cadence mixed Helm, dockerize defaults #1259

Closed pregnor closed 3 years ago

pregnor commented 3 years ago
Q A
Bug fix? yes
New feature? no
API breaks? no
Deprecations? no
Related tickets -
License Apache 2.0

What's in this PR?

Fixed Cadence mixed Helm, dockerize defaults.

Why?

To correct it.

Additional context

Helm YAML Go text/templates and jwilder/dockerize's docker YAML Go text/templates - which is used by Cadence internally - define the default function with different argument order. Helm defines it with default-value object argument order. jwilder/dockerize defines it with object default-value argument order.

We use both due to mixing Helm level templating and occasionally generating dockerize level templates into the docker configuration file.

I messed the mixing up when I was importing the changes of the new server version.

Checklist

To Do