Skip to content
This repository has been archived by the owner on Jun 19, 2024. It is now read-only.

Do not set INDEXER_CONFIG_PATH by default in Dockerfile #149

Merged
merged 2 commits into from
Feb 29, 2024

Conversation

joel-u410
Copy link
Contributor

This allows the docker image to be configured via env variables instead of a config file. (When INDEXER_CONFIG_PATH is set, the application will only use the file and will not attempt to read its config from env vars.) This is useful for environments like Google Cloud Run where it is simpler to set env vars than to mount a file into the container.

@joel-u410
Copy link
Contributor Author

I considered also removing the default RUST_LOG since some deployments also might not want that set. Should I do that here also?

@rllola
Copy link
Contributor

rllola commented Feb 28, 2024

Yeah sure @joel-u410 we can remove it.

@joel-u410
Copy link
Contributor Author

ok @rllola -- I'll push another commit to this branch for that.

@rllola rllola merged commit 3d4fdec into Zondax:main Feb 29, 2024
6 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants