-
Notifications
You must be signed in to change notification settings - Fork 3.9k
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.
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
(ec2): Unable to run integ.machine-image test #32017
Comments
Needs reproduction. Existing integ.machine-image.js.snapshot/integ-ec2-machine-image-test.template.json is using |
Reproducible when running
|
@phuhung273 Looks like it works with NOTE: As noted in Thanks, |
Thanks for your help @ashishdhingra that works. Seems like the current snapshot is dry-run, i will leave it as is. Hope we can improve it later not to confuse someone else. |
@phuhung273 Thanks for confirmation. Closing this issue. I will bring this to team's attention for guidance on fallback dummy regions for availability zone. |
Comments on closed issues and PRs are hard for our team to see. |
Another workaround is to set the region key in the stack creation
I'm not certain why the team decided to use dummy values for the AZ. Will dig into it. |
Describe the bug
Running
yarn integ test/aws-ec2/test/integ.machine-image.js --update-on-failed
return error:Resource handler returned message: "Value (test-region-1a) for parameter availabilityZone is invalid. Subnets can currently only be created in the following availability zones: us-east-1a, us-east-1b, us-east-1c, us-east-1d, us-east-1e, us-east-1f.
Regression Issue
Last Known Working CDK Version
No response
Expected Behavior
Integration test success
Current Behavior
Reproduction Steps
yarn integ test/aws-ec2/test/integ.machine-image.js --update-on-failed
Possible Solution
Use real region instead of
test
Additional Information/Context
Hi team, im new contributor. Please correct me if this issue is false alert or the way im testing is wrong.
While working on #32006, my change requires rerunning lots of integration tests, one of them is
integ.machine-image
Inspecting the snapshot f6c1e62#diff-6481037221541f788e8e52206ed56ff9f9d3948945623babf841d57f65c3cba2R21, find that all AZs in
template.json
are liketest-region-1a, test-region-2a
. Do we need any additional setup to run this test case ?CDK CLI Version
main
Framework Version
No response
Node.js Version
v20.18.0
OS
Dev container
Language
TypeScript
Language Version
No response
Other information
No response
The text was updated successfully, but these errors were encountered: