Skip to content

Conversation

trueleo
Copy link
Contributor

@trueleo trueleo commented Feb 22, 2023

No description provided.

@nitisht nitisht merged commit 414d2c9 into parseablehq:main Feb 23, 2023
@syepes
Copy link
Contributor

syepes commented Feb 23, 2023

@trueleo | @nitisht, Thanks for implementing this, its a great start.

Do you think we could also add some additional labels to add more context from where the alert is coming from?

impl CallableTarget for AlertManager {

Suggested additional labels:

https://prometheus.io/docs/prometheus/latest/configuration/configuration/#relabel_config __address__ = config.parseable.address __scheme__ = config.parseable.get_scheme() __alerts_path__ = /api/v1/logstream/{{stream_name}}/alert __meta_parseable_deployment_id = StorageMetadata.deployment_id __meta_parseable_version = StorageMetadata.version __meta_parseable_mode = StorageMetadata.mode __meta_parseable_alert_rule_type = "column" __meta_parseable_alert_rule_config_column = "status" __meta_parseable_alert_rule_config_operator = "notEqualTo" __meta_parseable_alert_rule_config_repeats = 2 __meta_parseable_alert_rule_config_value = 500 
@nitisht
Copy link
Member

nitisht commented Feb 24, 2023

Thanks @syepes this is very useful. Let me create an issue with details so we can track this properly.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants