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

Mention ./promtool check config prometheus.yml needed #2405

Open
Ricky-Tigg opened this issue Nov 24, 2023 · 0 comments
Open

Mention ./promtool check config prometheus.yml needed #2405

Ricky-Tigg opened this issue Nov 24, 2023 · 0 comments

Comments

@Ricky-Tigg
Copy link

Hello. Observations that apply to the whole documentation at prometheus.io:

  • By default the query in web browser for http://localhost:9090 opens at the Graph-named tab; which URL does imply http://localhost:9090/graph. Then the mention graph as part of the location path is unneeded.
  • Causing a modification to the Prometheus server's configuration file -prometheus.yml- does not come along with a recommendation of verification of the validity of its syntax. Knowing that at http://localhost:9090/config, no such verification is operated, this validation is left undone upon the current documentation. It would be worth rectifying this lack of instruction by including as a good practise such a verification, thus mentioning the related command ./promtool check config prometheus.yml. Illustration:
$ ./promtool check config prometheus.yml
Checking prometheus.yml
 SUCCESS: prometheus.yml is valid prometheus config file syntax
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

No branches or pull requests

1 participant