Configure the timescaledb image with our grafanareader user by default #39
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
lucyb
reviewed
Nov 21, 2023
lucyb
approved these changes
Nov 21, 2023
timescaledb-ha is the image built for use with kubernetes and was mistakenly used when timescaledb was added to the project in 3f4892e. This reduces the resulting image from ~3GB to ~700MB.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
When running locally we need a
grafanareaderuser set up in the timescaledb with appropriate permissions so we the TimescaleDBWriter class can create tables with the right permissions.If you've run the compose stack locally you'll need to remove the timescaledb volume to make it reinitialise. So you can test with:
This should run and stay up if all works. You can then double check it's working by running one of the ingestion commands, a single day of GitHub data is the fastest one.
My intention is to use this same setup to fix CI for #38.