rds: Add support for MariaDB RDS Proxy

[mavdi]

Describe the bug

Even though MariaDB support has been added to RDS Proxies, this is not reflected in CDK.

Expected Behavior

Given that MariaDB is supported in RDS Proxies, I'd like this to work.

https://aws.amazon.com/about-aws/whats-new/2022/07/amazon-rds-proxy-mariadb-version-10-support/

Current Behavior

Throws an error saying MariaDB support for proxies is not supported.

Error: Engine 'mariadb-10.6.16' does not support proxies
    at ProxyTarget.bind (/home/parallels/Documents/work/call4u-monorepo/infrastructure/node_modules/aws-cdk-lib/aws-rds/lib/proxy.js:1:2928)

Reproduction Steps

/ Create the RDS Proxy
    const rdsProxy = new aws_rds.DatabaseProxy(this, `${stackNamePrefix}-DatabaseProxy`, {
      proxyTarget: aws_rds.ProxyTarget.fromInstance(databaseInstance),
      secrets: [databaseSecret],
      vpc,
      securityGroups: [rdsProxySecurityGroup],
      requireTLS: false, // Optional: set to true to enforce TLS connections
    });

Possible Solution

No response

Additional Information/Context

No response

CDK CLI Version

2.131.0 (build 92b912d)

Framework Version

No response

Node.js Version

21.6.2

OS

Mac OS

Language

TypeScript

Language Version

No response

Other information

No response

[msambol]

I'll take this.

[msambol]

@mavdi Did you specify an engine family when creating databaseInstance ? Can I see the code for that ?

[pahud]

@msambol Yap I was just about to ask exactly the same question :-)

[mavdi]

Yes I did, see below:

    const databaseInstance = new aws_rds.DatabaseInstance(
      this,
      `${stackNamePrefix}-DatabaseInstance`,
      {
        securityGroups: [databaseSecurityGroup],
        engine: aws_rds.DatabaseInstanceEngine.mariaDb({
          version: aws_rds.MariaDbEngineVersion.VER_10_6_16,
        }),
        instanceType: aws_ec2.InstanceType.of(
          aws_ec2.InstanceClass.T3,
          aws_ec2.InstanceSize.MICRO
        ),
        vpc,
        vpcSubnets: {
          subnetType: aws_ec2.SubnetType.PUBLIC,
        },
        credentials: aws_rds.Credentials.fromSecret(databaseSecret),
        removalPolicy,
        publiclyAccessible: true,
        databaseName: "db",
        deletionProtection: false,
      }
    );

[msambol]

@mavdi / @pahud Looks like my PR is getting merged but I have a better fix coming for this. I'll open a new PR.

[github-actions]

⚠️COMMENT VISIBILITY WARNING⚠️

Comments on closed issues are hard for our team to see.
If you need more assistance, please either tag a team member or open a new issue that references this one.
If you wish to keep having a conversation with other community members under this issue feel free to do so.

[msambol]

@pahud Can you reopen this please?

[msambol]

@pahud This is a better fix: #29412. MariaDB needed the engine family added.