fix(ecs): Windows ECS Optimized AMI SSM parameter format is incorrect #26326
Conversation
|
|
github-actions
bot
added
the
beginning-contributor
There was a problem hiding this comment.
The pull request linter has failed. See the aws-cdk-automation comment below for failure reasons. If you believe this pull request should receive an exemption, please comment and provide a justification.
A comment requesting an exemption should contain the text Exemption Request. Additionally, if clarification is needed add Clarification Request to a comment.
|
I am from the Windows Containers Technologies team who owns the updating the public SSM param for Windows optimized AMIs. We have a customer who would like to get these params updated so they may be used in CDK. Can I get an Exemption Request from the integ tests? |
aws-cdk-automation
added
the
pr-linter/exemption-requested
The format of the Windows ECS optimized ssm parameter used by CDK is incorrect. The correct format can be found [here](https://docs.aws.amazon.com/AmazonECS/latest/developerguide/retrieve-ecs-optimized_windows_AMI.html) Closes aws#26327.
aws-cdk-automation
added
the
pr/needs-community-review
mrgrain
changed the title
github-actions
bot
added
the
effort/small
aws-cdk-automation
removed
the
pr/needs-community-review
mrgrain
added
the
pr-linter/exempt-integ-test
aws-cdk-automation
dismissed
their stale review
✅ Updated pull request passes all PRLinter validations. Dismissing previous PRLinter review.
|
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). |
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
|
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). |
…aws#26326) The format of the Windows ECS optimized ssm parameter used by CDK is incorrect. The correct format can be found [here](https://docs.aws.amazon.com/AmazonECS/latest/developerguide/retrieve-ecs-optimized_windows_AMI.html) ``` Current incorrect format: /aws/service/ecs/optimized-ami/windows_server/${this.windowsVersion}/english/full/recommended/image_id Correct format: /aws/service/ami-windows-latest/Windows_Server-${this.windowsVersion}-English-Full-ECS_Optimized ``` Closes aws#26327. ---- *By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
The format of the Windows ECS optimized ssm parameter used by CDK is incorrect. The correct format can be found here
Closes #26327.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license