Skip to content
This repository was archived by the owner on Mar 23, 2026. It is now read-only.

Skip test test_cfn_apigateway_swagger_import when running in K8s#13229

Merged
nik-localstack merged 1 commit intomainfrom
k8s/run-tests-for-community
Oct 7, 2025
Merged

Skip test test_cfn_apigateway_swagger_import when running in K8s#13229
nik-localstack merged 1 commit intomainfrom
k8s/run-tests-for-community

Conversation

@nik-localstack
Copy link
Copy Markdown
Contributor

@nik-localstack nik-localstack commented Oct 6, 2025 •
edited
Loading

Motivation

Fix failing tests from https://github.com/localstack/localstack-pro/pull/5371

Changes

  • Adds a marker in test_cfn_apigateway_swagger_import as it uses pytest http marker

@nik-localstack nik-localstack self-assigned this Oct 6, 2025
@nik-localstack nik-localstack added docs: skip Pull request does not require documentation changes notes: skip Pull request does not have to be mentioned in the release notes semver: patch Non-breaking changes which can be included in patch releases labels Oct 6, 2025
@nik-localstack nik-localstack added this to the 4.10 milestone Oct 6, 2025
@github-actions
Copy link
Copy Markdown

github-actions bot commented Oct 6, 2025 •
edited
Loading

Test Results - Preflight, Unit

22 292 tests  ±0   20 549 ✅ ±0   16m 2s ⏱️ +28s
     1 suites ±0    1 743 💤 ±0 
     1 files   ±0        0 ❌ ±0 

Results for commit 7f6a2ae. ± Comparison against base commit 928f470.

♻️ This comment has been updated with latest results.

@github-actions
Copy link
Copy Markdown

github-actions bot commented Oct 6, 2025 •
edited
Loading

Test Results (amd64) - Acceptance

7 tests  ±0   5 ✅ ±0   3m 30s ⏱️ +10s
1 suites ±0   2 💤 ±0 
1 files   ±0   0 ❌ ±0 

Results for commit 7f6a2ae. ± Comparison against base commit 928f470.

♻️ This comment has been updated with latest results.

@github-actions
Copy link
Copy Markdown

github-actions bot commented Oct 6, 2025 •
edited
Loading

LocalStack Community integration with Pro

  2 files  ±    0    2 suites  ±0   36m 7s ⏱️ - 1h 22m 56s
576 tests  - 4 219  461 ✅  - 3 998  115 💤  - 221  0 ❌ ±0 
578 runs   - 4 219  461 ✅  - 3 998  117 💤  - 221  0 ❌ ±0 

Results for commit 7f6a2ae. ± Comparison against base commit 928f470.

This pull request removes 4219 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]
…

♻️ This comment has been updated with latest results.

@github-actions
Copy link
Copy Markdown

github-actions bot commented Oct 6, 2025 •
edited
Loading

Test Results (amd64) - Integration, Bootstrap

  5 files  ±    0    5 suites  ±0   48m 23s ⏱️ - 1h 50m 10s
600 tests  - 4 569  486 ✅  - 4 187  114 💤  - 382  0 ❌ ±0 
606 runs   - 4 569  486 ✅  - 4 187  120 💤  - 382  0 ❌ ±0 

Results for commit 7f6a2ae. ± Comparison against base commit 928f470.

This pull request removes 4569 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]
…

♻️ This comment has been updated with latest results.

@github-actions
Copy link
Copy Markdown

github-actions bot commented Oct 6, 2025 •
edited
Loading

Test Results - Alternative Providers

575 tests   - 775   330 ✅  - 371   26m 20s ⏱️ - 13m 46s
  1 suites  -   4   245 💤  - 404 
  1 files    -   4     0 ❌ ±  0 

Results for commit 7f6a2ae. ± Comparison against base commit 928f470.

This pull request removes 775 tests. 
tests.aws.services.cloudwatch.test_cloudwatch.TestCloudWatchMultiProtocol ‑ test_basic_operations_multiple_protocols[json]
tests.aws.services.cloudwatch.test_cloudwatch.TestCloudWatchMultiProtocol ‑ test_basic_operations_multiple_protocols[query]
tests.aws.services.cloudwatch.test_cloudwatch.TestCloudWatchMultiProtocol ‑ test_basic_operations_multiple_protocols[smithy-rpc-v2-cbor]
tests.aws.services.cloudwatch.test_cloudwatch.TestCloudWatchMultiProtocol ‑ test_exception_serializing_with_no_shape_in_spec[json]
tests.aws.services.cloudwatch.test_cloudwatch.TestCloudWatchMultiProtocol ‑ test_exception_serializing_with_no_shape_in_spec[query]
tests.aws.services.cloudwatch.test_cloudwatch.TestCloudWatchMultiProtocol ‑ test_exception_serializing_with_no_shape_in_spec[smithy-rpc-v2-cbor]
tests.aws.services.cloudwatch.test_cloudwatch.TestCloudWatchMultiProtocol ‑ test_multi_protocol_client_fixture[json]
tests.aws.services.cloudwatch.test_cloudwatch.TestCloudWatchMultiProtocol ‑ test_multi_protocol_client_fixture[query]
tests.aws.services.cloudwatch.test_cloudwatch.TestCloudWatchMultiProtocol ‑ test_multi_protocol_client_fixture[smithy-rpc-v2-cbor]
tests.aws.services.cloudwatch.test_cloudwatch.TestCloudwatch ‑ test_alarm_lambda_target
…

♻️ This comment has been updated with latest results.

@nik-localstack nik-localstack force-pushed the k8s/run-tests-for-community branch from 15c96ad to 7f6a2ae Compare October 7, 2025 10:20
@nik-localstack nik-localstack changed the title Fix tests for running in K8s Skip test test_cfn_apigateway_swagger_import when running in K8s Oct 7, 2025
@nik-localstack nik-localstack marked this pull request as ready for review October 7, 2025 11:04
simonrw
simonrw approved these changes Oct 7, 2025
View reviewed changes
Copy link
Copy Markdown
Contributor

@simonrw simonrw left a comment

Choose a reason for hiding this comment

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

Thanks for adding this

@nik-localstack nik-localstack merged commit f6aa9a1 into main Oct 7, 2025
42 checks passed
@nik-localstack nik-localstack deleted the k8s/run-tests-for-community branch October 7, 2025 11:17
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@pinzon pinzon Awaiting requested review from pinzon

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

1 more reviewer

@simonrw simonrw simonrw approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@nik-localstack nik-localstack

Labels

docs: skip Pull request does not require documentation changes notes: skip Pull request does not have to be mentioned in the release notes semver: patch Non-breaking changes which can be included in patch releases

Projects

None yet

Milestone

4.10

Development

Successfully merging this pull request may close these issues.

2 participants

@nik-localstack @simonrw