aws-cloudformation / cloudformation-coverage-roadmap

The AWS CloudFormation Public Coverage Roadmap
https://aws.amazon.com/cloudformation/
Creative Commons Attribution Share Alike 4.0 International
1.11k stars 54 forks source link

Unable to add SageMaker Studio KernelGatewayApp LifeCycleConfig #1392

Open manshahe opened 1 year ago

manshahe commented 1 year ago

Name of the resource

AWS::SageMaker::Domain

Resource Name

No response

Issue Description

I want to add a LifeCycleConfig to the SageMaker Studio Domain KernelGatewayApp, however the CFN fails.

The ARN pattern for a LifeCycleConfig is documented here: https://docs.aws.amazon.com/AWSCloudFormation/latest/UserGuide/aws-properties-sagemaker-domain-resourcespec.html#cfn-sagemaker-domain-resourcespec-lifecycleconfigarn

Here is the code Snippet:

AWSTemplateFormatVersion: "2010-09-09"

Resources:
  StudioDomain:
    Type: AWS::SageMaker::Domain
    Properties:
      VpcId: "vpc-xxx"
      SubnetIds: ["subnet-xxx"]
      DomainName: "MyDomainName"
      AuthMode: SSO
      DefaultUserSettings:
        ExecutionRole: "arn:aws:iam::xxx:role/AmazonSageMaker-ExecutionRole-20210903T143299"
        KernelGatewayAppSettings:
          DefaultResourceSpec:
            LifecycleConfigArn: "arn:aws:sagemaker:eu-central-1:xxx:studio-lifecycle-config/test-config"

Expected Behavior

The Studio Domain with a LifeCycleConfig added to the KernelGatewayApp

Observed Behavior

CloudFormation errors out with the message:

Properties validation failed for resource StudioDomain with message: 

#/DefaultUserSettings/KernelGatewayAppSettings/DefaultResourceSpec/LifecycleConfigArn: failed validation constraint for keyword [pattern]

Test Cases

ARN for LifeCycleConfig: arn:aws:sagemaker:eu-central-1:xxx:studio-lifecycle-config/test-config Follows the pattern : arn:aws[a-z\-]*:sagemaker:[a-z0-9\-]*:[0-9]{12}:studio-lifecycle-config/.*

Other Details

No response

johanneslanger commented 1 year ago

+1

niklas-palm commented 1 year ago

+1

lavarith commented 1 year ago

+1

fabbazon commented 1 year ago

+1