Bug/fix support azure managed identities in Databricks operator#40332
Bug/fix support azure managed identities in Databricks operator#40332potiuk merged 3 commits intoapache:mainfrom
Conversation
|
Congratulations on your first Pull Request and welcome to the Apache Airflow community! If you have any issues or are unsure about any anything please check our Contributors' Guide (https://github.com/apache/airflow/blob/main/contributing-docs/README.rst)
|
86b3545 to
6f21caa
Compare
Lee-W
left a comment
There was a problem hiding this comment.
The changes look good, but we'll need to fix the failed tests and add some unit tests to this change
I'm working on fixing the tests. I hope to have it done today. |
No worries. 🙂 |
58e125b to
9408428
Compare
|
Awesome work, congrats on your first merged pull request! You are invited to check our Issue Tracker for additional contributions. |
…he#40332) * Add azure identity to get bearer token * Add azure-identity as an additional-extras dependency * Fix tests --------- Co-authored-by: Marcel Martinelli <marcel.martinelli@rabobank.nl>
Fixes: #38762
^ Add meaningful description above
Read the Pull Request Guidelines for more information.
In case of fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in a newsfragment file, named
{pr_number}.significant.rstor{issue_number}.significant.rst, in newsfragments.