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

feat: add filter capability to exclude events from being sent #376

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

woehrl01
Copy link

@woehrl01 woehrl01 commented Feb 19, 2024

Description

This adds a new feature to be able to filter events being sent to NewRelic. It uses cached expressions via https://github.com/expr-lang/expr for minimal performance impact.

New configuration fields are excludeEventsFilter and excludeDescribeFilter which recieves an list of expression strings.

Type of change

  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • New feature / enhancement (non-breaking change which adds functionality)
  • Security fix
  • Bug fix (non-breaking change which fixes an issue)

Checklist:

  • Add changelog entry following the contributing guide
  • Documentation has been updated
  • This change requires changes in testing:
    • unit tests
    • E2E tests

@woehrl01 woehrl01 requested a review from a team as a code owner February 19, 2024 07:44
@woehrl01
Copy link
Author

@dbudziwojskiNR I appreciate your feedback on this PR, thank you!

@juanjjaramillo
Copy link
Contributor

Thank you @woehrl01 for your submission. We will talk to our Product Manager to determine when we can review the PR.

@woehrl01 woehrl01 force-pushed the exclude_filter branch 5 times, most recently from 5d4eab2 to 5bd7073 Compare March 28, 2024 12:42
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.

None yet

2 participants