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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants