-
Notifications
You must be signed in to change notification settings - Fork 4.1k
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
(aws-appmesh): priority cannot be configured in gateway routes #16821
Comments
@desoss - thank you very much for reporting the issue and I am very sorry for the inconvenience you are facing. I am taking a looking a look at this issue we will respond by end of today with updates. |
I am confirming CDK is currently not allowing Gateway Route to configure I verified that the App Mesh console as well as AWS CLI does allow configuring Gateway Route with However, CloudFormation GAtewayRouteSpec does not contain a field for I am further getting confirmation if this is the bug/miss. |
@desoss - We've verified Meanwhile, as a workaround, please consider taking one of the below steps:
Again, we deeply apologize for the inconvenience. |
Thanks, @Seiya6329 . So that we can decide if it's worth investing some time in an AWS CLI script or not. |
@desoss - we are looking at 1.0 - 1.5 month to deploy all changes. |
Quick update: We are deploying changes to CloudFormation to include I think we are looking at end of November as a target for full CDK update completion. |
For CDK L2 support, we are still at end of November. |
Submitted the PR for supporting Gateway Route |
Adding the Gateway Route `Priority` support. This is not a new feature but it was missed from the implementation. The implementation method is mimicking how Route's `Priority` is implemented: - [route-spec.ts](https://github.com/aws/aws-cdk/blob/master/packages/%40aws-cdk/aws-appmesh/lib/route-spec.ts) - [route.ts](https://github.com/aws/aws-cdk/blob/master/packages/%40aws-cdk/aws-appmesh/lib/route.ts) Fixes #16821 ---- *By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
|
@desoss - we have pushed the support with the gateway route priority. This feature should become available on 12/16 (Thu). |
Adding the Gateway Route `Priority` support. This is not a new feature but it was missed from the implementation. The implementation method is mimicking how Route's `Priority` is implemented: - [route-spec.ts](https://github.com/aws/aws-cdk/blob/master/packages/%40aws-cdk/aws-appmesh/lib/route-spec.ts) - [route.ts](https://github.com/aws/aws-cdk/blob/master/packages/%40aws-cdk/aws-appmesh/lib/route.ts) Fixes aws#16821 ---- *By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
What is the problem?
Even if the AWS console allows to specify the gateway route priority in the gateway route configuration this is not possible in CDK.
Reproduction Steps
Here I've no way to specify the priority of a gateway route.
What did you expect to happen?
I expect to be able to configure the gateway route priority as in the console:
What actually happened?
Currently, the priority cannot be configured in the gateway routes.
CDK CLI Version
1.126.0
Framework Version
No response
Node.js Version
16.4.1
OS
macOS
Language
Typescript
Language Version
No response
Other information
No response
The text was updated successfully, but these errors were encountered: