bedrock: KnowledgeBaseConfigurationProperty is missing sqlKnowledgeBaseConfiguration property in latest CDK version 2.176.0

[nroshanr]

Describe the bug

KnowledgeBaseConfigurationProperty is missing the sqlKnowledgeBaseConfiguration property which is currently supported by CloudFormation AWS::Bedrock::KnowledgeBase KnowledgeBaseConfiguration

Regression Issue

• Select this option if this issue appears to be a regression.

Last Known Working CDK Version

No response

Expected Behavior

sqlKnowledgeBaseConfiguration should not be missing in L1 construct.

Current Behavior

CDK gives the following error:
Object literal may only specify known properties, and 'sqlKnowledgeBaseConfiguration' does not exist in type 'IResolvable | KnowledgeBaseConfigurationProperty'.

Reproduction Steps

Here's a sample code to replicate the error:

      new bedrock.CfnKnowledgeBase(this, 'knowledgebase', {
          knowledgeBaseConfiguration: {
            type: 'type',
            vectorKnowledgeBaseConfiguration: {
              embeddingModelArn: 'embeddingModelArn'
          },
          kendraKnowledgeBaseConfiguration: {
            kendraIndexArn: 'kendraIndexArn'
          }
          sqlKnowledgeBaseConfiguration:{
            type: 'REDSHIFT'
          }
        },
        roleArn: 'roleName',
        name: 'Name'
      })

Possible Solution

No response

Additional Information/Context

No response

CDK CLI Version

2.176.0

Framework Version

No response

Node.js Version

v18.20.5

OS

Linux

Language

TypeScript

Language Version

5.3.3

Other information

No response

[ashishdhingra]

@nroshanr Good morning. Thanks for opening the issue. I do see that schema file aws-bedrock-knowledgebase.json was updated 14 hrs ago. It could take a while for CDK to be updated with the new schema changes. Please wait for next release for sqlKnowledgeBaseConfiguration to be included in CfnKnowledgeBaseProps.

Thanks,
Ashish

[github-actions]

This issue has not received a response in a while. If you want to keep this issue open, please leave a comment below and auto-close will be canceled.

[PiyushVaishnavRB]

@ashishdhingra Is there any tentative date by which this release would happen?

[ashishdhingra]

Refer to PR #33019 which was merged yesterday and has below changes:

├[~] service aws-bedrock
│ └ resources
│    ├[~]  resource AWS::Bedrock::KnowledgeBase
│    │  ├ properties
│    │  │  └ KnowledgeBaseConfiguration: - KnowledgeBaseConfiguration (required, immutable)
│    │  │                                + KnowledgeBaseConfiguration (required)
│    │  └ types
│    │     ├[+]  type CuratedQuery
│    │     │  ├      documentation: Curated query or question and answer pair
│    │     │  │      name: CuratedQuery
│    │     │  └ properties
│    │     │     ├ NaturalLanguage: string (required)
│    │     │     └ Sql: string (required)
│    │     ├[~] type KnowledgeBaseConfiguration
│    │     │ └ properties
│    │     │    ├ KendraKnowledgeBaseConfiguration: - KendraKnowledgeBaseConfiguration
│    │     │    │                                   + KendraKnowledgeBaseConfiguration (immutable)
│    │     │    ├[+] SqlKnowledgeBaseConfiguration: SqlKnowledgeBaseConfiguration
│    │     │    ├ Type: - string (required)
│    │     │    │       + string (required, immutable)
│    │     │    └ VectorKnowledgeBaseConfiguration: - VectorKnowledgeBaseConfiguration
│    │     │                                        + VectorKnowledgeBaseConfiguration (immutable)

Please wait for next CDK release.

[github-actions]

This issue has not received a response in a while. If you want to keep this issue open, please leave a comment below and auto-close will be canceled.

[dave-moser]

Thank you for that update. SqlKnowledgeBaseConfiguration is now available in cdk version 2.177.0

It does create the Bedrock KnowledgeBase, however, it does not successfully bind the KB to the Redshift serverless cluster.

CDK Code:

    const kb = new cdk.aws_bedrock.CfnKnowledgeBase(
      this,
      "BedrockKnowledgeBaseRedshift",
      {
        name: props.knowledgeBaseName,
        roleArn: bedrockRole.roleArn,
        knowledgeBaseConfiguration: {
          type: "SQL",
          sqlKnowledgeBaseConfiguration: {
            type: "REDSHIFT",
            redshiftConfiguration: {
              queryEngineConfiguration: {
                serverlessConfiguration: {
                  authConfiguration: {
                    type: "IAM",
                  },
                  workgroupArn: props.workGroupArn,
                },
                type: "SERVERLESS",
              },
              queryGenerationConfiguration: {
                executionTimeoutSeconds: props.executionTimeoutSeconds,
                generationContext: {
                  curatedQueries: props.curatedQueries,
                  tables: props.tables,
                },
              },
              storageConfigurations: [
                {
                  redshiftConfiguration: {
                    databaseName: props.databaseName,
                  },
                  type: "REDSHIFT",
                },
              ],
            },
          },
        },
      }
    );

Console screenshot (no Query Engine created):

[moelasmar]

Thanks @dave-moser for raising this issue.
I believe the issue of being unable to bind the knowledge base to redshift is a service side issue, since as I understood from your reply that the missing properties are now exist in the L1 construct which is actually one to one mapping with the CFN resources, and CDK do not add any logic.

Could you please check the synthesized CFN template and confirm if this is what you expect.

[dave-moser]

I think you are correct. Other data sources require another CF construct to pass in more detail.

    new cdk.aws_bedrock.CfnDataSource(
      this,
      "BedrockKnowledgeBaseRedshiftDataSource",
      {
        name: `kb-ds`,
        knowledgeBaseId: kb.attrKnowledgeBaseId,
        dataSourceConfiguration: {
          type: "REDSHIFT_METADATA",
        },
      }
    );

This CF construct does not have parameters for Redshift. The above code throws an error.

Where do I report such an issue with CloudFormation?

FYI - I am able to successfully create the working KB with Redshift using the Boto3 API.

[moelasmar]

I doubled check the existing CFN DataSource Configuration .. the Type property seems supports Redshift, but there is no place to add more configuration. Please, check this issue with CFN team.

[dave-moser]

Thanks. I have added the issue here.

aws-cloudformation/cloudformation-coverage-roadmap#2207