inveniosoftware / helm-invenio

Helm charts for deploying an Invenio instance
https://helm-invenio.readthedocs.io
7 stars 19 forks source link

INVENIO_DATACITE_ENABLED is "false" instead of "False" leading to a bad python evaluation #53

Closed avivace closed 1 year ago

avivace commented 1 year ago

We are using .Values.invenio.datacite.enabled both as a flag (e.g. to decide when to inject the datacite credentials) and as the value to be injected as INVENIO_DATACITE_ENABLED in the invenio config map (which feeds the environment). However, YAML's boolean false gets evaluated to true in python, leading to a messed up configuration (e.g. try to deploy the helm charts with the default values, datacite comes up as enabled because INVENIO_DATACITE_ENABLED is set as false and not False in the configMap).

avivace commented 1 year ago

Option 1:

Option 2:

avivace commented 1 year ago

Currently waiting for some feedback - should this be escalated to invenio-config and how we parse invenio.cfg?

kpsherva commented 1 year ago

could it be because of the quote marks around the value? it gets evaluated as string and then as true. Did you test without the quotes?