Skip to content

[aws] Add max_number_of_messages to AWS integration config #2299

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

Merged
merged 1 commit into from
Dec 7, 2021
Merged

[aws] Add max_number_of_messages to AWS integration config #2299

merged 1 commit into from
Dec 7, 2021

Conversation

andrewkroh
Copy link
Member

@andrewkroh andrewkroh commented Dec 1, 2021

What does this PR do?

Give users the ability to set max_number_of_messages for the aws-s3 input.

https://www.elastic.co/guide/en/beats/filebeat/current/filebeat-input-aws-s3.html#_max_number_of_messages

Checklist

  • I have reviewed tips for building integrations and this pull request is aligned with them.
  • I have verified that all data streams collect metrics or logs.
  • I have added an entry to my package's changelog.yml file.
  • I have verified that Kibana version constraints are current according to guidelines.

Related issues

Screenshots

Screen Shot 2021-12-01 at 5 12 17 PM

@elasticmachine
Copy link

Pinging @elastic/security-external-integrations (Team:Security-External Integrations)

@elasticmachine
Copy link

Pinging @elastic/integrations (Team:Integrations)

@andrewkroh andrewkroh changed the title Add max_number_of_messages to AWS integration config [aws] Add max_number_of_messages to AWS integration config Dec 1, 2021
@andrewkroh andrewkroh requested a review from a team December 1, 2021 22:11
@elasticmachine
Copy link

elasticmachine commented Dec 1, 2021

💚 Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS
Pipeline View Test View Changes Artifacts preview preview

Expand to view the summary

Build stats

  • Start Time: 2021-12-06T18:24:30.220+0000

  • Duration: 26 min 42 sec

  • Commit: d58a41c

Test stats 🧪

Test Results
Failed 0
Passed 303
Skipped 0
Total 303

🤖 GitHub comments

To re-run your PR in the CI, just comment with:

  • /test : Re-trigger the build.

@andrewkroh andrewkroh merged commit d554f7e into elastic:master Dec 7, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request Integration:aws AWS Team:Integrations Label for the Integrations team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[aws] Expose max_number_of_messages config in cloudtrail data stream
3 participants