-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
[Filebeat] Add Cloudflare R2 to provider list #32620
Conversation
/test |
Pinging @elastic/security-external-integrations (Team:Security-External Integrations) |
The S3 input doesn't have a codeowner listed, and I see changes from both cloud monitoring and SEI in the history. Requesting review from those teams, and we should clarify who the codeowner for this module should be if we can. |
This pull request does not have a backport label.
|
Can this be merged? |
Can someone from @elastic/obs-cloud-monitoring approve if they want to support this? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
* Add Cloudflare R2 to provider list in AWS S3 input * update unit tests * update changelog
What does this PR do?
Now that Cloudflare R2 is out of closed beta, this adds their domain to the provider list to automatically detect and set the provider field.
Why is it important?
Checklist
CHANGELOG.next.asciidoc
orCHANGELOG-developer.next.asciidoc
.Author's Checklist
How to test this PR locally
Related issues
Use cases
Screenshots
Logs