Skip to content

Conversation

@thesuperzapper
Copy link
Member

@thesuperzapper thesuperzapper commented Mar 29, 2022

What issues does your PR fix?

What does your PR do?

  • Added values to set externalDatabase/externalRedis passwords directly with plain-text values:
    • externalDatabase.password
    • externalRedis.password
  • Added values to set externalDatabase username from a secret:
    • externalDatabase.userSecret
    • externalDatabase.userSecretKey (default: postgresql-user)
  • Added warning to NOTES.txt if user sets externalDatabase/externalRedis using the plain-text values.

Checklist

For all Pull Requests

For releasing ONLY

@thesuperzapper thesuperzapper added this to the airflow-8.6.0 milestone Mar 29, 2022
@thesuperzapper thesuperzapper added the status/ready-to-merge status - this will be merged into next release label Mar 29, 2022
@thesuperzapper thesuperzapper changed the title feat: set db password with value + db user from secret feat: database passwords with values + username from secret Mar 31, 2022
@thesuperzapper thesuperzapper merged commit b4f2b98 into airflow-helm:main Mar 31, 2022
@thesuperzapper thesuperzapper deleted the external-database-user-secret branch April 1, 2022 08:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

status/ready-to-merge status - this will be merged into next release

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

specify externalDatabase username from Secret (improves PostgresOperator support)

1 participant