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

Wrong port number for node manager agents in prometheus/prometheus.consul.yml.template when using Docker #2277

Closed
jeftadlvw opened this issue May 1, 2024 · 0 comments · Fixed by #2282
Labels
bug Something isn't working right

Comments

@jeftadlvw
Copy link

Installation details

  • Scylla-Monitoring Version: master
  • Scylla-Version: latest (5.4.6)
  • Toy cluster with local docker containers

When provisioning a prometheus configuration using

./prometheus-config.sh -L <manager ip> --compose

and starting the monitoring stack with docker compose, Prometheus is unable to connect to each node's manager agent. This is because the consul-template sets the wrong port for the manager agent:

I used the Dockerfile provided by the documentation to create an image that contains a manager agent (see https://manager.docs.scylladb.com/stable/docker/index.html). In that image, the default port for the manager is 5090. So it should be the default port in the configuration too (instead of 56090).

IMO, using the -L together with --compose should also be documented.

Useful information:

  • Source code references are on commit ba445e1c0458741d377b08c16b45d05f61b98279, which, at the time of writing, is the latest commit to master
@jeftadlvw jeftadlvw added the bug Something isn't working right label May 1, 2024
@jeftadlvw jeftadlvw changed the title Wrong port number for node manager agents in prometheus/prometheus.consul.yml.template Wrong port number for node manager agents in prometheus/prometheus.consul.yml.template when using Docker May 1, 2024
@amnonh amnonh added this to the Monitoring 4.7.2 milestone May 5, 2024
amnonh added a commit to amnonh/scylla-grafana-monitoring that referenced this issue May 8, 2024
amnonh added a commit that referenced this issue May 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working right
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants