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

DDS: F5 Distributed Cloud integration (V2.0.0) #2457

Open
wants to merge 21 commits into
base: master
Choose a base branch
from

Conversation

madhavpandya-crest
Copy link

@madhavpandya-crest madhavpandya-crest commented Aug 2, 2024

What does this PR do?

PR for a integration F5 Distributed Cloud 2.0.0

Motivation

-- OOTB detection rules JSON would be shared separately with the required teams as a part of separate repository .
-- During the standard attribute remapping, we preserve certain source attributes to ensure compatibility with older versions without causing conflicts. For other fields, we do not preserve them as per suggested best practices. As a result, filters using these standard attributes will populate values from other integrations, following the current datadog behavior.

Review checklist

  • PR has a meaningful title or PR has the no-changelog label attached
  • Feature or bugfix has tests
  • Git history is clean
  • If PR impacts documentation, docs team has been notified or an issue has been opened on the documentation repo
  • If this PR includes a log pipeline, please add a description describing the remappers and processors.

Additional Notes

  • The system currently does not monitor DDoS events. This functionality will be added once relevant event data becomes available.
  • The pipeline is encountering failures due to source conflicts. This issue needs to be resolved to ensure smooth pipeline execution.
  • There is no impact on the existing F5 Integration. As discussed over email, the current pipeline implementation will not affect the access logs.

@madhavpandya-crest madhavpandya-crest marked this pull request as ready for review August 7, 2024 13:29
@madhavpandya-crest madhavpandya-crest requested review from a team as code owners August 7, 2024 13:29
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.

2 participants