Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ogc_api_records should have env var for credentials stores to db #319

Open
jeanmi151 opened this issue Aug 24, 2023 · 3 comments
Open

ogc_api_records should have env var for credentials stores to db #319

jeanmi151 opened this issue Aug 24, 2023 · 3 comments
Assignees

Comments

@jeanmi151
Copy link
Contributor

here : https://github.com/georchestra/datadir/blob/docker-master/geonetwork/microservices/ogc-api-records/config.yml#L23

@fvanderbiest
Copy link
Member

fvanderbiest commented Aug 24, 2023

Good catch !
Let's fix this (and backport on 23.x)

@edevosc2c
Copy link
Member

edevosc2c commented Oct 11, 2023

Just stumbled upon this issue too now with isère.

Do we agree to have the envs inside the config.yml instead of overriding it using an environment variable like on k8s: https://github.com/georchestra/helm-georchestra/blob/main/templates/geonetwork/ogc-api-records/ogc-api-records-deployment.yaml#L63C11-L64C111?

Will also need to use envs-database-georchestra file for the service "ogc-api-records"

@fvanderbiest
Copy link
Member

Sounds like a fair option

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants