kaxil commented on pull request #8640: URL: https://github.com/apache/airflow/pull/8640#issuecomment-623027707
> Small white lie from the above testing: in order to make it work, I had to have the following patch applied. Not sure what's going on there...maybe my pip install is wrong or `vault.py` is out of date or something. > > ```diff > diff --git a/airflow/providers/hashicorp/secrets/vault.py b/airflow/providers/hashicorp/secrets/vault.py > index c0fd23581..27058e9c4 100644 > --- a/airflow/providers/hashicorp/secrets/vault.py > +++ b/airflow/providers/hashicorp/secrets/vault.py > @@ -24,7 +24,7 @@ import hvac > from cached_property import cached_property > from hvac.exceptions import InvalidPath, VaultError > > -from airflow import AirflowException > +from airflow.exceptions import AirflowException > from airflow.secrets import BaseSecretsBackend > from airflow.utils.log.logging_mixin import LoggingMixin > > ``` > > Seems unrelated to the k8s support though. Happy for you to make that change too. It will avoid cycles. ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected]
