KEYVAULT_URI takes precedence over AIRFLOW_VAR_KEYVAULT_URI

Asbjørn L. Johansenrequested to merge
asbjorjo/fix-azure-credentials-test into master

Type of change

  • Bug Fix
  • Feature

Does this introduce a change in the core logic?

  • [No]

Does this introduce a change in the cloud provider implementation, if so which cloud?

  • AWS
  • Azure
  • GCP
  • IBM

Updates description?

Previously the AIRFLOW_VAR_KEYVAULT_URI environment variable was set by the test for Azure credentials. In the test environment for the master branch the KEYVAULT_URI environment variable is set and this takes precedence. Changed to set KEYVAULT_URI environment variable in test mock. This avoids the mock environment being overridden by the environment the test is running in.

Edited by Asbjørn L. Johansen

Merge request reports