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

allow to override the default PrometheusURL in alerts #716

Closed
gesellix opened this Issue May 22, 2015 · 8 comments

Comments

Projects
None yet
3 participants
@gesellix
Copy link
Contributor

gesellix commented May 22, 2015

When running Prometheus in a Docker container and producing alerts, they contain a GeneratorURL looking like this: http://b640ba8f91c1:9090//graph#.... The b640ba8f91c1 hash is a generated Docker container hostname. Obviously, such hostnames are useless in the actual alerts (or in the Alertmanager /alerts overview).

Would it make sense to add a configuration option setting the desired external hostname? When empty, the current mechanism could be used.

@brian-brazil

This comment has been minimized.

Copy link
Member

brian-brazil commented May 22, 2015

I think this is a good idea.

@gesellix

This comment has been minimized.

Copy link
Contributor Author

gesellix commented May 22, 2015

@brian-brazil should I provide a PR?

@brian-brazil

This comment has been minimized.

Copy link
Member

brian-brazil commented May 22, 2015

If you can that'd be great.

@juliusv

This comment has been minimized.

Copy link
Member

juliusv commented May 22, 2015

@gesellix

This comment has been minimized.

Copy link
Contributor Author

gesellix commented May 22, 2015

@juliusv proposed flag name: web.hostname ok?

@juliusv

This comment has been minimized.

Copy link
Member

juliusv commented May 22, 2015

@gesellix Sounds good.

@gesellix

This comment has been minimized.

Copy link
Contributor Author

gesellix commented May 23, 2015

resolved with #717

@gesellix gesellix closed this May 23, 2015

simonpasquier pushed a commit to simonpasquier/prometheus that referenced this issue Oct 12, 2017

Merge pull request prometheus#716 from prometheus/grobie/checksums
Render tarball sha256 checksum if available for release
@lock

This comment has been minimized.

Copy link

lock bot commented Mar 24, 2019

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@lock lock bot locked and limited conversation to collaborators Mar 24, 2019

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
You can’t perform that action at this time.