bug: fix name of appConfig configmap

[coufalja]

Description of the change

Scope the name of generated config map to reflect potential name overrides, currently it is impossible to install 2 instances in a single namespace.

Existing or Associated Issue(s)

N/A

Checklist

• Chart version bumped in Chart.yaml according to semver.
• Variables are documented in the values.yaml and added to the README.md. The helm-docs utility can be used to generate the necessary content. Use helm-docs --dry-run to preview the content.
• JSON Schema generated.
• List tests pass for Chart using the Chart Testing tool and the ct lint command.

[coufalja]

I guess the change in readme.md are caused by the newer version of helm-docs

[tumido]

Hi @coufalja,

Thanks for contributing. I think the fix itself is good, however you'll need to regenerate the README either via pre-commit or via helm-docs with the following arguments:

charts/.pre-commit-config.yaml

Lines 6 to 12 in 4625407

	args:
	# Make the tool search for charts only under the ``charts` directory
	- --chart-search-root=charts
	# The `./` makes it relative to the chart-search-root set above
	- --template-files=./_templates.gotmpl
	# A base filename makes it relative to each chart directory found
	- --template-files=README.md.gotmpl

(Btw, an interesting coincidence, we share the same surname and we live in the same city, are we related? 😄 )

[coufalja]

Got it, thanks for pointing that out 👍

[coufalja]

(Btw, an interesting coincidence, we share the same surname and we live in the same city, are we related? 😄 )

I don't think we are but interesting coincidence nevertheless 😄

[tumido]

/lgtm
/approve