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

RGTA/EC2: skip RGTA EC2 test in Pro#13743

Merged
bentsku merged 1 commit into
mainfrom
skip-rgta-ec2
Feb 11, 2026
Merged

RGTA/EC2: skip RGTA EC2 test in Pro#13743
bentsku merged 1 commit into
mainfrom
skip-rgta-ec2

Conversation

@bentsku
Copy link
Copy Markdown
Contributor

@bentsku bentsku commented Feb 11, 2026

Motivation

As we don't have EC2 yet implemented in the new version of RGTA, we need to skip the tests. The way to skip is not ideal, but it works 👍 this was taken from tests.aws.services.stepfunctions.v2.services.test_ecs_task_service.TestTaskServiceECS

Changes

  • skip EC2 RGTA test

Tests

Related

@bentsku bentsku added this to the 4.14 milestone Feb 11, 2026
@bentsku bentsku self-assigned this Feb 11, 2026
@bentsku bentsku requested a review from aidehn as a code owner February 11, 2026 11:20
@bentsku bentsku added semver: patch Non-breaking changes which can be included in patch releases aws:resourcegroupstaggingapi AWS Resource Groups Tagging docs: skip Pull request does not require documentation changes notes: skip Pull request does not have to be mentioned in the release notes labels Feb 11, 2026
@github-actions
Copy link
Copy Markdown

LocalStack Community integration with Pro

2 files  2 suites   42s ⏱️
3 tests 0 ✅ 3 💤 0 ❌
5 runs  0 ✅ 5 💤 0 ❌

Results for commit c9bfc6d.

@github-actions
Copy link
Copy Markdown

Test Results - Preflight, Unit

23 140 tests  ±0   21 276 ✅ ±0   6m 22s ⏱️ +3s
     1 suites ±0    1 864 💤 ±0 
     1 files   ±0        0 ❌ ±0 

Results for commit c9bfc6d. ± Comparison against base commit 7839bdf.

@bentsku bentsku added the review: merge when ready Signals to the reviewer that a PR can be merged if accepted label Feb 11, 2026
@github-actions
Copy link
Copy Markdown

Test Results (amd64) - Acceptance

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

Results for commit c9bfc6d. ± Comparison against base commit 7839bdf.

@github-actions
Copy link
Copy Markdown

Test Results (amd64) - Integration, Bootstrap

 5 files   5 suites   9m 7s ⏱️
27 tests 24 ✅ 3 💤 0 ❌
33 runs  24 ✅ 9 💤 0 ❌

Results for commit c9bfc6d.

Copy link
Copy Markdown
Contributor

@aidehn aidehn left a comment

Choose a reason for hiding this comment

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

Ah this slipped through on my migration PR😭 Thank you for fixing this quickly 🙏🏻

Copy link
Copy Markdown
Collaborator

@k-a-il k-a-il left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@bentsku bentsku merged commit 17e97cc into main Feb 11, 2026
63 checks passed
@bentsku bentsku deleted the skip-rgta-ec2 branch February 11, 2026 12:01
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

aws:resourcegroupstaggingapi AWS Resource Groups Tagging docs: skip Pull request does not require documentation changes notes: skip Pull request does not have to be mentioned in the release notes review: merge when ready Signals to the reviewer that a PR can be merged if accepted 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