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

Use compose to start the monitoring stack #2123

Merged
merged 3 commits into from Dec 13, 2023
Merged

Conversation

amnonh
Copy link
Collaborator

@amnonh amnonh commented Dec 3, 2023

This series adds an experimental option to use docker compose to start the monitoring stack.
the --compose command line option to start all will create a compose file and a .env file.

That can be used with docker-compose up to start the stack.

Fixes #1073

@tzach
Copy link
Contributor

tzach commented Dec 3, 2023

@amnonh I expected to see the doc part of this PR, including adding compose as a prereqieust

@amnonh
Copy link
Collaborator Author

amnonh commented Dec 3, 2023

@amnonh I expected to see the doc part of this PR, including adding compose as a prereqieust

Done, please have a look

@amnonh amnonh added this to the Monitoring 4.6 milestone Dec 3, 2023
;;
(--auto-restart) DOCKER_PARAM="$DOCKER_PARAM restart: unless-stopped\n"
;;
(--victoria-metrics) VICTORIA_METRICS="1"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I wonder if this still works?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It will not under compose, I'm not testing it with our regular monitoring stack, as it's all experimental, I wanted to start with an initial commit and make changes if I'll see that it's in use.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've added victoria metrics support

@amnonh amnonh merged commit e33b0ce into scylladb:master Dec 13, 2023
1 check passed
@amnonh amnonh deleted the compose branch December 13, 2023 11:50
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.

Move the configuration to a setup part
3 participants