-
Notifications
You must be signed in to change notification settings - Fork 9.1k
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
Add TLS and basic authentication #8316
Conversation
Cc @SuperQ |
Nice, do we want to mark this as experimental for the first release? |
The flag in the node_exporter is --web.config, the config file in prometheus is --config.file. Do we want to fix the exporter-toolkit to use --web.config.file ? |
Maybe |
I realise that we probably want to add in the toolkit a way to validate the config early and fail before launching the tsdb. The toolkit will also need to take the certificate relatively to the config file, like in Prometheus. |
Pending prometheus/exporter-toolkit#22 |
94e3f76
to
b1e55d4
Compare
This is ready for a second round of reviews. cc @beorn7 for awareness, as you are sheperd for next week release. |
Yeah, would be great to get this in. Remember that the planned date for cutting the RC is already in 2 days… |
Avoid starting up components like the TSDB if we can't bind to the web listening port. Signed-off-by: Ben Kochie <superq@gmail.com>
589fd77
to
7cc857a
Compare
Updated and ready for a new review. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
I think you might need a rebase.
Signed-off-by: Julien Pivotto <roidelapluie@inuits.eu>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
No description provided.