fix: #222

[akash-jain-10]

What this PR does / why we need it :

Fixes #222

To test the above changes, run the below commands -

cd charts/openmetadata
helm template om . --values values.yaml --set openmetadata.config.secretsManager.provider=managed-azure-kv --set openmetadata.config.secretsManager.additionalParameters.enabled=true

cd charts/openmetadata
helm template om . --values values.yaml --set openmetadata.config.secretsManager.provider=managed-aws --set openmetadata.config.secretsManager.additionalParameters.enabled=true

Type of change :

• Bug fix
• Improvement
• New feature
• Documentation

Checklist:

• I have read the CONTRIBUTING document.
• Chart Version bumped
• Variables are documented in the README.md
• Title of the PR starts with chart name
• My code follows the style guidelines of this project
• I have performed a self-review of my code
• I have commented my code, particularly in hard-to-understand areas
• I have made corresponding changes to the documentation
• All new and existing tests passed.

Reviewers

[Nemeczek]

Hey, quick question. in the docs there is option to do "If only the , parameter is provided, we will use Azure's default Authentication Credential." in the Airflow part. I saw that in the OM Server there is support for DefaultAzureCredential as well. So will be able, after this PR is merged, to do either/both of ServicePrincipal with ClientId and Secret or using ManagedIndeity by providing only KV name on both server and airflow?

[tutte]

LGTM! 🚀