charts icon indicating copy to clipboard operation
charts copied to clipboard

Recognize iam auth database login

Open kishubhatta opened this issue 3 years ago • 5 comments

Is this a request for help?: No


Is this a BUG REPORT or FEATURE REQUEST? (choose one): BUG REPORT

Version of Helm and Kubernetes: NA

Which chart: jfrog/artifactory

What happened: Unused values can crash the artifactory pod

What you expected to happen: Values not needed are not required

How to reproduce it (as minimally and precisely as possible): Deploy the chart using Google SQL IAM auth

Anything else we need to know:

kishubhatta avatar Jan 17 '22 21:01 kishubhatta

Hi @kishubhatta ,

Can you kindly add some details about your use case and why you are seeking JFrog to implement support for IAM database authentication?

peleduri avatar May 19 '22 15:05 peleduri

@peleduri you don't need to support IAM db auth. You only need to gracefully handle the case where the helm value for password is not specified. Currently, the artifactory application simply crashes if one does not specify password value. One needs to set some dummy value, even when using IAM auth and cloud sql proxy sidecar.

kishubhatta avatar May 20 '22 05:05 kishubhatta

@kishubhatta can you please share the reproducible steps

chukka avatar May 24 '22 07:05 chukka

@chukka it is in the description.

kishubhatta avatar May 25 '22 18:05 kishubhatta

This works already. I'm using a cloud-sql-proxy sidecar and then just passing in db-user (no password) to connect with the IAM GCP service account.

Although, it might be a nice feature to use the CloudSQL JDBC Socket Factory in place of the MySQL driver. No sidecar needed then.

johnjelinek avatar Jan 26 '24 02:01 johnjelinek