mongodb-query-exporter icon indicating copy to clipboard operation
mongodb-query-exporter copied to clipboard

Make `checksum/hash annotations` work with `existingConfig`

Open VannTen opened this issue 2 years ago • 1 comments

Is your feature request related to a problem? Please describe

When using the existingConfig value, the checksum/hash annotations seems not to be computed with respect to the used config.

Describe the solution you'd like

Adapt the tempalte in order to works either with embedded config or with externally provided

Additional context

  • I'm using this chart as a dependant chart for another one, and the config is templated in the parent chart. I do that since some of the configMap depend on my environments, and the chart values are used in the configmap templating

Describe alternatives you've considered

  • Better: allow to use values variables inside the values file. I'm not an expert of writing helm templates, but it seems to be possible with special treatment in the templates.
  • My current workaround : provide the annotations myself in podAnnotations.

This is not critical as the workaround is very simple, but I think it would make the chart more 'out of the box' usable.

If I have some time next week I might try to work on that.

VannTen avatar May 13 '22 13:05 VannTen

Hum, in fact. the podAnnotations workaround does not work yet. I reuse the sha256sum but it's not interpreted as a template and hence it's stay at it's litteral value in the rendered manifest... :thinking:

VannTen avatar May 13 '22 13:05 VannTen