Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Make AWS region name configurable for tests #9070

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 2 commits into from
Sep 6, 2023

Conversation

joe4dev
Copy link
Member

@joe4dev joe4dev commented Sep 5, 2023

Motivation

We should have a configuration to change the test region. For cross-account tests, we should be able to configure the region in order to test cross-account tests within the same region.

Discovered during testing of the Lambda invocation loop rework #8970
/cc @dfangl @dominikschubert

Changes

Makes the AWS region name configurable in tests.

@joe4dev joe4dev added the semver: patch Non-breaking changes which can be included in patch releases label Sep 5, 2023
@joe4dev joe4dev self-assigned this Sep 5, 2023
@joe4dev
Copy link
Member Author

joe4dev commented Sep 5, 2023

We could also consider dropping the _NAME suffix to match the AWS_REGION config 🤔

@github-actions
Copy link

github-actions bot commented Sep 5, 2023

LocalStack Community integration with Pro

       2 files         2 suites   1h 31m 5s ⏱️
2 166 tests 1 679 ✔️ 487 💤 0
2 167 runs  1 679 ✔️ 488 💤 0

Results for commit 952a547.

♻️ This comment has been updated with latest results.

Copy link
Member

@viren-nadkarni viren-nadkarni left a comment

Choose a reason for hiding this comment

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

🚀

Removing the suffix `_NAME` aligns the environment variable naming with the AWS CLI using `AWS_REGION` (which overwrites `AWS_DEFAULT_REGION`).
https://docs.aws.amazon.com/cli/latest/userguide/cli-configure-envvars.html
@joe4dev
Copy link
Member Author

joe4dev commented Sep 6, 2023

We could also consider dropping the _NAME suffix to match the AWS_REGION config 🤔

Align environment variable naming with AWS CLI using TEST_AWS_REGION without the _NAME suffix.

Removing the suffix _NAME aligns the environment variable naming with the AWS CLI using AWS_REGION (which overwrites AWS_DEFAULT_REGION).
https://docs.aws.amazon.com/cli/latest/userguide/cli-configure-envvars.html

@joe4dev
Copy link
Member Author

joe4dev commented Sep 6, 2023

Flaky arm64 test unrelated and reported in flaky test list (already know flake)

@joe4dev joe4dev merged commit 7afdbab into master Sep 6, 2023
@joe4dev joe4dev deleted the make-aws-region-name-configurable-for-tests branch September 6, 2023 14:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
semver: patch Non-breaking changes which can be included in patch releases
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants