-
Notifications
You must be signed in to change notification settings - Fork 53
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
Cert & alerting fixes #1793
Cert & alerting fixes #1793
Conversation
8631853
to
45963db
Compare
45963db
to
6323bd1
Compare
This has a breaking change with evaluating Prometheus based alert status. Prometheus rules may show up as |
7985428
to
2ce8d10
Compare
@@ -16,4 +18,5 @@ type AlertingDriverOptions struct { | |||
InternalRoutingKey string `option:"internalRoutingKey"` | |||
AlertingOptions *shared.AlertingClusterOptions `option:"alertingOptions"` | |||
Subscribers []chan alertingClient.AlertingClient `option:"subscribers"` | |||
TlsConfig *tls.Config `option:"tlsConfig"` |
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 really don't wanna be that guy but acronyms should be in consistent case -> TLSConfig
Ok
status instead ofPending