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

Skip to content

Fix MA/MR pipeline for EC2 tests #12672

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 2 commits into from
May 28, 2025
Merged

Fix MA/MR pipeline for EC2 tests #12672

merged 2 commits into from
May 28, 2025

Conversation

anisaoshafi
Copy link
Contributor

Motivation

Fix MA/MR pipeline for EC2 tests for describe_availability_zones: failure.

Changes

Fix region name in the aws client.
Add transformer for the region.

@anisaoshafi anisaoshafi added aws:ec2 Amazon Elastic Compute Cloud semver: patch Non-breaking changes which can be included in patch releases labels May 28, 2025
@github-actions GitHub Actions
Copy link

LocalStack Community integration with Pro

 2 files  ±    0   2 suites  ±0   49s ⏱️ - 1h 44m 35s
31 tests  - 4 437  29 ✅  - 4 051  2 💤  - 386  0 ❌ ±0 
33 runs   - 4 437  29 ✅  - 4 051  4 💤  - 386  0 ❌ ±0 

Results for commit daa0340. ± Comparison against base commit 433aeff.

This pull request removes 4437 tests. 
tests.aws.scenario.bookstore.test_bookstore.TestBookstoreApplication ‑ test_lambda_dynamodb
tests.aws.scenario.bookstore.test_bookstore.TestBookstoreApplication ‑ test_opensearch_crud
tests.aws.scenario.bookstore.test_bookstore.TestBookstoreApplication ‑ test_search_books
tests.aws.scenario.bookstore.test_bookstore.TestBookstoreApplication ‑ test_setup
tests.aws.scenario.kinesis_firehose.test_kinesis_firehose.TestKinesisFirehoseScenario ‑ test_kinesis_firehose_s3
tests.aws.scenario.lambda_destination.test_lambda_destination_scenario.TestLambdaDestinationScenario ‑ test_destination_sns
tests.aws.scenario.lambda_destination.test_lambda_destination_scenario.TestLambdaDestinationScenario ‑ test_infra
tests.aws.scenario.loan_broker.test_loan_broker.TestLoanBrokerScenario ‑ test_prefill_dynamodb_table
tests.aws.scenario.loan_broker.test_loan_broker.TestLoanBrokerScenario ‑ test_stepfunctions_input_recipient_list[step_function_input0-SUCCEEDED]
tests.aws.scenario.loan_broker.test_loan_broker.TestLoanBrokerScenario ‑ test_stepfunctions_input_recipient_list[step_function_input1-SUCCEEDED]
…

@anisaoshafi anisaoshafi added this to the 4.5 milestone May 28, 2025
@github-actions GitHub Actions
Copy link

Test Results - Preflight, Unit

21 579 tests  ±0   19 927 ✅ ±0   6m 6s ⏱️ -10s
     1 suites ±0    1 652 💤 ±0 
     1 files   ±0        0 ❌ ±0 

Results for commit daa0340. ± Comparison against base commit 433aeff.

@github-actions GitHub Actions
Copy link

Test Results (MA/MR) - Preflight, Unit

21 579 tests  ±0   19 927 ✅ ±0   6m 56s ⏱️ +42s
     1 suites ±0    1 652 💤 ±0 
     1 files   ±0        0 ❌ ±0 

Results for commit daa0340. ± Comparison against base commit 433aeff.

@github-actions GitHub Actions
Copy link

Test Results (amd64) - Acceptance

7 tests  ±0   5 ✅ ±0   3m 8s ⏱️ -1s
1 suites ±0   2 💤 ±0 
1 files   ±0   0 ❌ ±0 

Results for commit daa0340. ± Comparison against base commit 433aeff.

@github-actions GitHub Actions
Copy link

Test Results (amd64, MA/MR) - Acceptance

7 tests  ±0   5 ✅ ±0   3m 14s ⏱️ -1s
1 suites ±0   2 💤 ±0 
1 files   ±0   0 ❌ ±0 

Results for commit daa0340. ± Comparison against base commit 433aeff.

@github-actions GitHub Actions
Copy link

Test Results - Alternative Providers

597 tests  ±0   420 ✅ ±0   14m 39s ⏱️ -13s
  4 suites ±0   177 💤 ±0 
  4 files   ±0     0 ❌ ±0 

Results for commit daa0340. ± Comparison against base commit 433aeff.

@github-actions GitHub Actions
Copy link

Test Results (amd64, MA/MR) - Integration, Bootstrap

 5 files  ±    0   5 suites  ±0   8m 19s ⏱️ - 2h 13m 48s
53 tests  - 4 770  51 ✅  - 4 228  2 💤  - 539  0 ❌  - 3 
59 runs   - 4 770  51 ✅  - 4 228  8 💤  - 539  0 ❌  - 3 

Results for commit daa0340. ± Comparison against base commit 433aeff.

This pull request removes 4770 tests. 
tests.aws.scenario.bookstore.test_bookstore.TestBookstoreApplication ‑ test_lambda_dynamodb
tests.aws.scenario.bookstore.test_bookstore.TestBookstoreApplication ‑ test_opensearch_crud
tests.aws.scenario.bookstore.test_bookstore.TestBookstoreApplication ‑ test_search_books
tests.aws.scenario.bookstore.test_bookstore.TestBookstoreApplication ‑ test_setup
tests.aws.scenario.kinesis_firehose.test_kinesis_firehose.TestKinesisFirehoseScenario ‑ test_kinesis_firehose_s3
tests.aws.scenario.lambda_destination.test_lambda_destination_scenario.TestLambdaDestinationScenario ‑ test_destination_sns
tests.aws.scenario.lambda_destination.test_lambda_destination_scenario.TestLambdaDestinationScenario ‑ test_infra
tests.aws.scenario.loan_broker.test_loan_broker.TestLoanBrokerScenario ‑ test_prefill_dynamodb_table
tests.aws.scenario.loan_broker.test_loan_broker.TestLoanBrokerScenario ‑ test_stepfunctions_input_recipient_list[step_function_input0-SUCCEEDED]
tests.aws.scenario.loan_broker.test_loan_broker.TestLoanBrokerScenario ‑ test_stepfunctions_input_recipient_list[step_function_input1-SUCCEEDED]
…

@github-actions GitHub Actions
Copy link

Test Results (MA/MR) - Alternative Providers

597 tests  ±0   420 ✅ ±0   14m 44s ⏱️ -20s
  4 suites ±0   177 💤 ±0 
  4 files   ±0     0 ❌ ±0 

Results for commit daa0340. ± Comparison against base commit 433aeff.

viren-nadkarni
viren-nadkarni approved these changes May 28, 2025
View reviewed changes
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.

Sorry for failing to spot this earlier. Looks good!

@github-actions GitHub Actions
Copy link

Test Results (amd64) - Integration, Bootstrap

    5 files  ±0      5 suites  ±0   2h 21m 30s ⏱️ - 1m 20s
4 823 tests ±0  4 282 ✅ ±0  541 💤 ±0  0 ❌ ±0 
4 829 runs  ±0  4 282 ✅ ±0  547 💤 ±0  0 ❌ ±0 

Results for commit daa0340. ± Comparison against base commit 433aeff.

@anisaoshafi anisaoshafi merged commit a298730 into master May 28, 2025
12 of 13 checks passed
@anisaoshafi anisaoshafi deleted the fix-ma-mr-for-ec2-tests branch May 28, 2025 09:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@viren-nadkarni viren-nadkarni viren-nadkarni approved these changes

@macnev2013 macnev2013 Awaiting requested review from macnev2013 macnev2013 is a code owner

Assignees

@viren-nadkarni viren-nadkarni

Labels
aws:ec2 Amazon Elastic Compute Cloud semver: patch Non-breaking changes which can be included in patch releases
Projects
None yet
Milestone
4.5
Development

Successfully merging this pull request may close these issues.
2 participants
@anisaoshafi @viren-nadkarni