Fix(redshift): add missing db_user parameter for IAM auth

[synthpieter]

Problem

SQLMesh's Redshift connection configuration was missing the db_user parameter, which is required by the redshift_connector library when using IAM authentication.

When enabling iam (iam: True) in the connection config it throws, understandably so, an InterfaceError

redshift_connector.error.InterfaceError: Connection parameter db_user must be specified when using IAM authentication

However, it was not allowed to add this parameter to the config.

Error: Invalid 'redshift' connection config:
  Invalid field 'db_user':
    Extra inputs are not permitted

Verify your config.yaml and environment variables.

So iam authentication was not really useable.

Solution

Added the db_user field to the RedshiftConnectionConfig class in three places:

1. Documentation - Added parameter description in the docstring and docs
2. Field Declaration - Added db_user: t.Optional[str] = None to the class
3. Connection Kwargs - Added "db_user" to the _connection_kwargs_keys property
4. Tests - Added db_user to corresponding test.

Usage

With this fix, you can now configure Redshift IAM authentication in config.yaml:

gateways:
  redshift:
    connection:
      type: redshift
      database: your_database
      host: your-cluster.region.redshift.amazonaws.com
      port: 5439
      cluster_identifier: your-cluster
      iam: True
      db_user: your_db_username

Fixes #2146

Disclaimer

This is my first contribution to an open source project ever, so please let me know if this is the right way of doing things :)
I locally give the fix a spin and it seems to fix the problem!

[CLAassistant]

All committers have signed the CLA.

[StuffbyYuki]

@synthpieter it looks like there is some conflict with the base branch. Would you mind looking into revolving that?

[synthpieter]

@StuffbyYuki I just fixed the conflict! To be honest there was something going on with a test in the pipeline before and tried to fix that, but honestly couldn't make it work at the time. That looks gone though!

[StuffbyYuki]

@synthpieter k it looks like your commits need DCO checks. Could you add them?

"All proposed commits should include a Signed-off-by: line in their commit message. This is most conveniently done by using --signoff (-s) when running git commit."

[StuffbyYuki]

Once everything looks good I'll merge this PR!

[StuffbyYuki]

@synthpieter Just following up on my last comment!

[StuffbyYuki]

@synthpieter Another follow up here

[synthpieter]

@StuffbyYuki Sorry for the delay, I'm not actively working with SQLMesh at the moment, so the urgency for my own fix was gone ;)

I think (and hope) I fixed it right!