Skip to content

[AWS] Add SQS API calls documentation and required S3 permissions #12072

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 0 commits into from
Dec 12, 2024
Merged

[AWS] Add SQS API calls documentation and required S3 permissions #12072

merged 0 commits into from
Dec 12, 2024

Conversation

@gpop63
Copy link
Contributor

@gpop63 gpop63 commented Dec 11, 2024

Overview

This PR adds SQS API calls documentation and required S3 permissions:

  • Documents SQS ReceiveMessage, DeleteMessage and ChangeMessageVisibility API calls
  • Adds s3:ListBucket and s3:GetBucketLocation permissions requirements

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.
  • I have verified that any added dashboard complies with Kibana's Dashboard good practices

Author's Checklist

  • [ ]

How to test this PR locally

Related issues

Screenshots
@andrewkroh andrewkroh added Integration:aws AWS enhancement New feature or request labels Dec 11, 2024
@gpop63 gpop63 self-assigned this Dec 11, 2024
@gpop63 gpop63 added Team:Obs-InfraObs Observability Infrastructure Monitoring team [elastic/obs-infraobs-integrations] documentation Improvements or additions to documentation. Applied to PRs that modify *.md files. labels Dec 11, 2024
@elastic-vault-github-plugin-prod
Copy link

elastic-vault-github-plugin-prod bot commented Dec 11, 2024 •
edited
Loading

🚀 Benchmarks report

To see the full report comment with /test benchmark fullreport
@gpop63 gpop63 marked this pull request as ready for review December 11, 2024 16:43
@gpop63 gpop63 requested review from a team as code owners December 11, 2024 16:43
@gpop63 gpop63 requested a review from agithomas December 11, 2024 16:44
@andrewkroh andrewkroh added the Team:obs-ds-hosted-services Observability Hosted Services team [elastic/obs-ds-hosted-services] label Dec 11, 2024
@agithomas agithomas requested a review from alaudazzi December 11, 2024 16:46
andrewkroh
andrewkroh reviewed Dec 11, 2024
View reviewed changes
@elastic-sonarqube
Copy link

Quality Gate passed Quality Gate passed

Issues
0 New issues
0 Fixed issues
0 Accepted issues

Measures
0 Security Hotspots
100.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube
@elasticmachine
Copy link

💚 Build Succeeded

History

  • 💚 Build #19357 succeeded 40a048b36f0cb7f2b015c2c9333ebe9b8f3f2a81
  • 💔 Build #19356 failed c6c554a1e4f9fd430006e0d044e099518b2ca50f

cc @gpop63
@agithomas agithomas mentioned this pull request Dec 12, 2024
Closed
agithomas
agithomas approved these changes Dec 12, 2024
View reviewed changes
Copy link
Contributor

@agithomas agithomas left a comment

Choose a reason for hiding this comment

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

LGTM!
alaudazzi
alaudazzi approved these changes Dec 12, 2024
View reviewed changes
Copy link
Contributor

@alaudazzi alaudazzi left a comment

Choose a reason for hiding this comment

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

LGTM
@gpop63 gpop63 merged commit 21ca46b into elastic:main Dec 12, 2024
5 checks passed
@gpop63 gpop63 deleted the aws-improve-docs branch December 12, 2024 14:41
@elastic-vault-github-plugin-prod
Copy link

Package aws - 2.36.1 containing this change is available at https://epr.elastic.co/package/aws/2.36.1/
harnish-crest-data pushed a commit to chavdaharnish/integrations that referenced this pull request Feb 4, 2025
@gpop63
[AWS] Add SQS API calls documentation and required S3 permissions (el…
3e531d6 
…astic#12072) * add SQS calls and S3 permissions in docs * bump package version * fix pr id * add SQS GetQueueAttributes sort permissions
harnish-crest-data pushed a commit to chavdaharnish/integrations that referenced this pull request Feb 5, 2025
@gpop63
[AWS] Add SQS API calls documentation and required S3 permissions (el…
14ebf59 
…astic#12072) * add SQS calls and S3 permissions in docs * bump package version * fix pr id * add SQS GetQueueAttributes sort permissions
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation. Applied to PRs that modify *.md files. enhancement New feature or request Integration:aws AWS Team:obs-ds-hosted-services Observability Hosted Services team [elastic/obs-ds-hosted-services] Team:Obs-InfraObs Observability Infrastructure Monitoring team [elastic/obs-infraobs-integrations]

5 participants

@gpop63 @elasticmachine @andrewkroh @alaudazzi @agithomas