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

Add support for ignoring IPs by ASN in riemann-http #295

Merged
merged 1 commit into from
Jun 29, 2024
Merged

Conversation

smortex
Copy link
Member

@smortex smortex commented Jun 19, 2024

Monitoring a website behind a Content Delivery Network (CDN) may lead to
flapping metrics when the short-lived IP addresses where the service is
accessible change.

Allow to provide a list of Autonomous System Numbers (ASN) that we can
ignore for well-known CDN service providers. Use the MaxMind ASN
database provided by the user for IP lookups.

This is not a hard dependency as no ASN filtering is done by default, so
only add this dependency for testing and assume the end-user will handle
the soft requirement on his own if they want to do filter-out some ASN.

@smortex smortex added the enhancement New feature or request label Jun 19, 2024
@smortex smortex force-pushed the filter-by-asn branch 2 times, most recently from b3f2c84 to ce01a18 Compare June 19, 2024 21:34
jamtur01
jamtur01 previously approved these changes Jun 29, 2024
Copy link
Member

@jamtur01 jamtur01 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lgtm

@smortex smortex dismissed jamtur01’s stale review June 29, 2024 10:46

The merge-base changed after approval.

Monitoring a website behind a Content Delivery Network (CDN) may lead to
flapping metrics when the short-lived IP addresses where the service is
accessible change.

Allow to provide a list of Autonomous System Numbers (ASN) that we can
ignore for well-known CDN service providers.  Use the MaxMind ASN
database provided by the user for IP lookups.

This is not a hard dependency as no ASN filtering is done by default, so
only add this dependency for testing and assume the end-user will handle
the soft requirement on his own if they want to do filter-out some ASN.
Copy link
Member

@jamtur01 jamtur01 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@jamtur01 jamtur01 merged commit c4ef790 into main Jun 29, 2024
7 checks passed
@smortex smortex deleted the filter-by-asn branch July 2, 2024 16:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants