Skip to content

chore(codeguruprofiler): add region constraints to integ tests#37172

Merged
mergify[bot] merged 2 commits intoaws:mainfrom
aemada-aws:fix/codeguruprofiler-integ-tests
Mar 12, 2026
Merged

chore(codeguruprofiler): add region constraints to integ tests#37172
mergify[bot] merged 2 commits intoaws:mainfrom
aemada-aws:fix/codeguruprofiler-integ-tests

Conversation

@aemada-aws
Copy link
Copy Markdown
Collaborator

Issue # (if applicable)

Closes #.

Reason for this change

The aws-codeguruprofiler integration tests fail when deployed to regions where the AWS::CodeGuruProfiler::ProfilingGroup CloudFormation resource type is not available (e.g. eu-west-3, sa-east-1, ap-northeast-2, ap-south-1, ca-central-1). The error is:

ValidationError: Template format error: Unrecognized resource types: [AWS::CodeGuruProfiler::ProfilingGroup]

Description of changes

Added regions constraint to the IntegTest props in both CodeGuruProfiler integ tests, restricting deployment to regions where the CFN resource type is confirmed available:

  • us-east-1, us-west-2, eu-west-1, eu-central-1, ap-northeast-1, ap-southeast-1, ap-southeast-2

No other code changes. Test logic and assertions are unchanged.

Files modified:

  • test/aws-codeguruprofiler/test/integ.profiler-group-import-functions.ts
  • test/aws-codeguruprofiler/test/integ.profiler-group.ts

Describe any new or updated permissions being added

N/A

Description of how you validated changes

yarn integ \
  test/aws-codeguruprofiler/test/integ.profiler-group-import-functions.js \
  test/aws-codeguruprofiler/test/integ.profiler-group.js \
  --disable-update-workflow --update-on-failed --force \
  --parallel-regions us-east-1 \
  --parallel-regions us-west-2 \
  --parallel-regions eu-west-1 \
  --parallel-regions eu-central-1 \
  --parallel-regions ap-northeast-1 \
  --parallel-regions ap-southeast-1 \
  --verbose

Result: Tests: 2 passed, 2 total

  • integ.profiler-group — SUCCESS (us-east-1, 83.6s)
  • integ.profiler-group-import-functions — SUCCESS (us-west-2, 163.9s)

Checklist


By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license

AWS::CodeGuruProfiler::ProfilingGroup CFN resource type is not available
in all regions. Added region constraints to both integ tests to restrict
deployment to supported regions.

Supported regions: us-east-1, us-west-2, eu-west-1, eu-central-1,
ap-northeast-1, ap-southeast-1, ap-southeast-2
@github-actions github-actions Bot added the p2 label Mar 5, 2026
@aws-cdk-automation aws-cdk-automation requested a review from a team March 5, 2026 12:51
@mergify mergify Bot added the contribution/core This is a PR that came from AWS. label Mar 5, 2026
@aemada-aws aemada-aws changed the title fix(codeguruprofiler): add region constraints to integ tests chore(codeguruprofiler): add region constraints to integ tests Mar 5, 2026
@mergify mergify Bot temporarily deployed to automation March 5, 2026 12:51 Inactive
@aemada-aws aemada-aws marked this pull request as ready for review March 5, 2026 12:51
@mergify mergify Bot temporarily deployed to automation March 5, 2026 12:51 Inactive
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Mar 5, 2026

⚠️ Experimental Feature: This security report is currently in experimental phase. Results may include false positives and the rules are being actively refined.
Please try merge from main to avoid findings unrelated to the PR.


TestsPassed ✅SkippedFailed
Security Guardian Results24 ran24 passed
TestResult
No test annotations available

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Mar 5, 2026

⚠️ Experimental Feature: This security report is currently in experimental phase. Results may include false positives and the rules are being actively refined.
Please try merge from main to avoid findings unrelated to the PR.


TestsPassed ✅SkippedFailed
Security Guardian Results with resolved templates24 ran24 passed
TestResult
No test annotations available

@aws-cdk-automation aws-cdk-automation added the pr/needs-maintainer-review This PR needs a review from a Core Team Member label Mar 5, 2026
@leonmk-aws leonmk-aws self-assigned this Mar 12, 2026
@mergify
Copy link
Copy Markdown
Contributor

mergify Bot commented Mar 12, 2026

Thank you for contributing! Your pull request will be updated from main and then merged automatically (do not update manually, and be sure to allow changes to be pushed to your fork).

@mergify
Copy link
Copy Markdown
Contributor

mergify Bot commented Mar 12, 2026

Merge Queue Status

  • Entered queue2026-03-12 13:09 UTC · Rule: default-squash
  • 🚫 Left the queue2026-03-12 13:09 UTC · at ad6f7906919383f36309018618d30e24e57d2a0a

This pull request spent 5 seconds in the queue, with no time running CI.

Reason

The pull request can't be updated

For security reasons, Mergify can't update this pull request. Try updating locally.
GitHub response: refusing to allow a GitHub App to create or update workflow .github/workflows/integration-test-deployment-auto.yml without workflows permission

Hint

You should update or rebase your pull request manually. If you do, this pull request will automatically be requeued once the queue conditions match again.
If you think this was a flaky issue, you can requeue the pull request, without updating it, by posting a @mergifyio queue comment.

@aws-cdk-automation aws-cdk-automation removed the pr/needs-maintainer-review This PR needs a review from a Core Team Member label Mar 12, 2026
@mergify
Copy link
Copy Markdown
Contributor

mergify Bot commented Mar 12, 2026

Thank you for contributing! Your pull request will be updated from main and then merged automatically (do not update manually, and be sure to allow changes to be pushed to your fork).

@mergify
Copy link
Copy Markdown
Contributor

mergify Bot commented Mar 12, 2026

Thank you for contributing! Your pull request will be updated from main and then merged automatically (do not update manually, and be sure to allow changes to be pushed to your fork).

@mergify mergify Bot merged commit 1299dba into aws:main Mar 12, 2026
23 of 24 checks passed
@mergify
Copy link
Copy Markdown
Contributor

mergify Bot commented Mar 12, 2026

Merge Queue Status

  • Entered queue2026-03-12 14:04 UTC · Rule: default-squash
  • Checks passed · in-place
  • Merged2026-03-12 14:05 UTC · at 706b86ffa9a38d41d93245aa76f290855e16f894

This pull request spent 10 seconds in the queue, with no time running CI.

Required conditions to merge

@github-actions
Copy link
Copy Markdown
Contributor

Comments on closed issues and PRs are hard for our team to see.
If you need help, please open a new issue that references this one.

@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Mar 12, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

contribution/core This is a PR that came from AWS. p2

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants