Add support for elk docker container environment variables #40
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
I think these are the only env vars that really matter for someone who wants the base functionality of elk. The only one that is default set is
NUXT_STORAGE_DRIVER
which is default set in the container to fs.I also made sure to support someone using their own k8s secret to make sure any future env vars are easy to add before we fully support them here. To see a full list of env vars that may or may not work (this is alpha software afterall), check out elk-zone/elk:
.env.example
These are all the values I've added:
This PR is in a draft state till I can test it locally, but wanted to get something out there sooner than later.
Kind regards and let me know if there's anything you'd like to improve here 🙏