Skip to content

Add IAM Auth support - #326

Open
bvalente wants to merge 3 commits into
percona:mainfrom
bvalente:percona-iam
Open

Add IAM Auth support#326
bvalente wants to merge 3 commits into
percona:mainfrom
bvalente:percona-iam

Conversation

@bvalente

Copy link
Copy Markdown

Related to prometheus upstream PR prometheus#983 and issue prometheus#749.

We're adding the option to authenticate to AWS RDS with IAM, as described in this guide: https://docs.aws.amazon.com/AmazonRDS/latest/UserGuide/UsingWithRDS.IAMDBAuth.Connecting.Go.html

To test, I used the following config file (with redacted values):

[client]
user=my-user
host=mydatabase.eu-central-1.rds.amazonaws.com
port=3306
aws-iam-auth=true
aws-region=eu-central-1
enable-cleartext-plugin=true
ssl-ca=global-bundle.pem

Signed-off-by: Bernardo Valente <bernardofvalente@gmail.com>
@bvalente
bvalente requested review from a team, BupycHuk and JiriCtvrtka and removed request for a team October 20, 2025 09:03
@it-percona-cla

it-percona-cla commented Oct 20, 2025

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@bvalente

Copy link
Copy Markdown
Author

Hello @JiriCtvrtka thank you for your review. Is there a schedule or something else needed to merge the PR? 😊

@bvalente

Copy link
Copy Markdown
Author

Hi again @JiriCtvrtka ! Would it be posible to go ahead with merging this PR? Please let me know if you need any more information or feedback.

@JiriCtvrtka
JiriCtvrtka requested a review from a team as a code owner August 19, 2026 12:31
@JiriCtvrtka
JiriCtvrtka requested review from 4nte and ademidoff and removed request for a team and BupycHuk August 19, 2026 12:31
@JiriCtvrtka

Copy link
Copy Markdown

@bvalente Yeah, sorry, I completely missed this one. I’m going to add some tests for the newly added fields, then ask QA whether we can proceed or if it needs to be tested manually.

@JiriCtvrtka

JiriCtvrtka commented Aug 20, 2026

Copy link
Copy Markdown

@bvalente I dont have permissions to push into your branch. I added test there: 9b6b5b9. Feel free to cherrypick this commit.

@bvalente

Copy link
Copy Markdown
Author

@JiriCtvrtka Thank you for the test files. I've cherry-picked your commit into my branch

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants