Skip to content

Updates the staging stack to to allow multiple accounts to read from S3#5830

Merged
dlvenable merged 1 commit into
opensearch-project:mainfrom
dlvenable:5796-testing-accounts
Jun 25, 2025
Merged

Updates the staging stack to to allow multiple accounts to read from S3#5830
dlvenable merged 1 commit into
opensearch-project:mainfrom
dlvenable:5796-testing-accounts

Conversation

@dlvenable
Copy link
Copy Markdown
Member

Description

Updates the staging stack to allow multiple accounts to access the read permissions. This will allow us to grant access to the OpenSearch CI test accounts.

Issues Resolved

Contributes toward #5796.

Check List

  • New functionality includes testing.
  • New functionality has a documentation issue. Please link to it in this PR.
    • New functionality has javadoc added
  • Commits are signed with a real name per the DCO

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

…ad permissions. This will allow us to grant access to the OpenSearch CI test accounts. Contributes toward opensearch-project#5796.

Signed-off-by: David Venable <dlv@amazon.com>
Copy link
Copy Markdown
Member

@gaiksaya gaiksaya left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks!

@dlvenable dlvenable merged commit 76d2d80 into opensearch-project:main Jun 25, 2025
44 of 47 checks passed
@dlvenable dlvenable deleted the 5796-testing-accounts branch June 25, 2025 21:38
JonahCalvo pushed a commit to JonahCalvo/os-data-prepper that referenced this pull request Jul 17, 2025
…ad permissions. This will allow us to grant access to the OpenSearch CI test accounts. Contributes toward opensearch-project#5796. (opensearch-project#5830)

Signed-off-by: David Venable <dlv@amazon.com>
Signed-off-by: Jonah Calvo <caljonah@amazon.com>
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.

3 participants