You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2020/06/10 15:12:50 UTC

[GitHub] [airflow] n4rk0o commented on pull request #9212: Add namespace/verify parameter in Hashicorp Vault provider

n4rk0o commented on pull request #9212:
URL: https://github.com/apache/airflow/pull/9212#issuecomment-642074979


   > HEllo @n4rk0o - there is a change in progress to extract Hashicorp Vault client and introduce Hook to it - I think your change should be rebased on top of i: #8974 . Hey @kaxil -> I think I addressed all the issues there, so maybe we should finally merge it.
   
   Hello @potiuk, I think it is a good idea to isolate the client in a specific Hook. I can be usefull for other usages. 
   What can I do to merge my proposal to yours?


----------------------------------------------------------------
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:
users@infra.apache.org