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

Prometheus persist: Programatically adds a docker volume on to the docker-compose to be used for storing prometheus data #49

Merged
merged 2 commits into from
Aug 2, 2024

Conversation

mikirov
Copy link
Contributor

@mikirov mikirov commented Aug 2, 2024

Programatically add a docker volume to the docker-compose to be used for storing prometheus data

@mikirov mikirov force-pushed the feat/persist-prometheus branch from 795d010 to 2b0fb27 Compare August 2, 2024 08:48
@mikirov mikirov force-pushed the feat/persist-prometheus branch from 2b0fb27 to 6236074 Compare August 2, 2024 08:50
@mikirov mikirov changed the title Programatically add a docker volume to the docker-compose to be used for storing prometheus data Prometheus persist: Programatically adds a docker volume on to the docker-compose to be used for storing prometheus data Aug 2, 2024
@mikirov mikirov self-assigned this Aug 2, 2024
@mikirov mikirov marked this pull request as ready for review August 2, 2024 08:54
@ltitanb
Copy link
Collaborator

ltitanb commented Aug 2, 2024

made some adjustments as discussed.

The prometheus data dir is now using a docker volume, as it avoids requiring the user to set up permissions for the folder. If the user needs to specify a different directory , then it's likely they have a different instance of prometheus running. In the future we could consider adding a "clean" command that removes all data created by commit boost, including the prometheus-data volume

@ltitanb ltitanb merged commit a5f4dca into main Aug 2, 2024
4 checks passed
@ltitanb ltitanb deleted the feat/persist-prometheus branch August 2, 2024 13:06
@ltitanb ltitanb mentioned this pull request Aug 2, 2024
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

Successfully merging this pull request may close these issues.

2 participants