Skip to content

ci: switch integration-test API Gateways to REGIONAL endpoints - #101

Open
RaphaelManke wants to merge 2 commits into
masterfrom
perf/regional-apigateway-endpoints
Open

ci: switch integration-test API Gateways to REGIONAL endpoints#101
RaphaelManke wants to merge 2 commits into
masterfrom
perf/regional-apigateway-endpoints

Conversation

@RaphaelManke

Copy link
Copy Markdown
Contributor

Summary

  • Every PR's integration-test CDK stack creates 17 apigateway.LambdaRestApi instances, which default to the EDGE endpoint type
  • EDGE REST APIs share a hard, non-adjustable 120-per-region account quota
  • With as few as 7 PRs concurrently open (each holding its stack until merge/close), the account hits the ceiling and every subsequent PR's Deploy CDK Stack job fails with This operation would breach the limit of 120 apis for EDGE endpoint type -- unrelated to that PR's actual changes
  • Switches all three LambdaRestApi call sites (integration-tests-stack.ts, python-tracing-scenarios-stack.ts, node-tracing-scenarios-stack.ts) to REGIONAL endpoints, which serve the same test purpose (API Gateway proxy integration, HTTP semconv extraction) without the shared ceiling

Test plan

  • cdk synth IntegrationTestsStack succeeds; confirmed via the synthesized templates that all 17 REST APIs per stack now specify REGIONAL and none remain EDGE
  • CI deploy for this PR succeeds end-to-end (confirms the stack still creates/tears down cleanly with the new endpoint type)

EDGE-type REST APIs share a hard 120-per-region account quota. Each
PR's integration-test stack creates 17 of them, so as few as 7
concurrently open PRs exhausts the quota and fails every subsequent
PR's deploy with "This operation would breach the limit of 120 apis
for EDGE endpoint type" -- unrelated to the PR's actual changes.
REGIONAL endpoints serve the same test purpose without that shared
ceiling.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants