-
Notifications
You must be signed in to change notification settings - Fork 4k
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
(device-farm): Support for device farm constructs #17893
Comments
It looks like the problem is that the AWS CloudFormation resource specification is regional https://docs.aws.amazon.com/AWSCloudFormation/latest/UserGuide/cfn-resource-specification.html but CDK only imports from us-east-1: https://github.com/aws/aws-cdk/blob/main/packages/%40aws-cdk/cfnspec/build-tools/update.sh#L60 But AWS DeviceFarm service is only available in us-west-2 and therefore only part of the us-west-2 resource specification file. |
By comparing the different regional files it turns out only Device Farm resources are affected:
All other resources in any other region are also part of the us-east-1 resource specification. |
Oh that's interesting. I guess this reflects an implicit assumption that products launch in us-east-1, and as such it's a framework-wide issue with the CDK. I've lodged #22565 to address this. |
Some AWS services are only in us-west-2 region, while CDK uses the AWS CloudFormation resource specification from us-east-1 for generating L1. This change adds suppport for resource specifications from additional regions, with an explicit allow list for services supported for this region. Fixes #17893 Fixes #22565 ---- *By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
|
Description
The AWS::DeviceFarm Cloudformation resources appear not to be exposed in the CDK at all.
This is a request for L1 constructs for those resources.
Use Case
To manage Device Farm resources via the CDK.
Proposed Solution
L1 Device Farm constructs.
Other information
No response
Acknowledge
The text was updated successfully, but these errors were encountered: