Enable TLS Certificate Verification for K8s Token Exchange in Databricks Hook - #63704
Merged
Conversation
eladkal
approved these changes
Mar 16, 2026
Contributor
Author
Added unit tests |
potiuk
approved these changes
Mar 16, 2026
Member
|
Pending unit tests fixing of course :) |
Contributor
Author
Thanks! fixed tests, we should be good now |
alexott
approved these changes
Mar 16, 2026
Contributor
If you seek for spelling error in docs build logs: |
jscheffl
approved these changes
Mar 16, 2026
jscheffl
left a comment
jscheffl
left a comment
Contributor
There was a problem hiding this comment.
Small spelling nit, else LGTM!
Contributor
Author
spelling fixed |
Contributor
|
Now just fighting against CI erros we had earlier... |
Contributor
Author
There is just one last error, but it does not seem to be related to my changes: |
eladkal
approved these changes
Mar 17, 2026
fat-catTW
pushed a commit
to fat-catTW/airflow
that referenced
this pull request
Mar 22, 2026
…icksHook` (apache#63704) * Enable TLS Certificate Verification for K8s Token Exchange * added unit tests * fixed tests * refactor tests * added change log * fixed spelling typo * updated changelog --------- Co-authored-by: Jens Scheffler <95105677+jscheffl@users.noreply.github.com> Co-authored-by: Jarek Potiuk <jarek@potiuk.com>
techcodie
pushed a commit
to techcodie/airflow
that referenced
this pull request
Mar 23, 2026
…icksHook` (apache#63704) * Enable TLS Certificate Verification for K8s Token Exchange * added unit tests * fixed tests * refactor tests * added change log * fixed spelling typo * updated changelog --------- Co-authored-by: Jens Scheffler <95105677+jscheffl@users.noreply.github.com> Co-authored-by: Jarek Potiuk <jarek@potiuk.com>
abhijeets25012-tech
pushed a commit
to abhijeets25012-tech/airflow
that referenced
this pull request
Apr 9, 2026
…icksHook` (apache#63704) * Enable TLS Certificate Verification for K8s Token Exchange * added unit tests * fixed tests * refactor tests * added change log * fixed spelling typo * updated changelog --------- Co-authored-by: Jens Scheffler <95105677+jscheffl@users.noreply.github.com> Co-authored-by: Jarek Potiuk <jarek@potiuk.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
5 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Enable Databricks hook TLS verification when communicating with the Kubernetes API server during federated token exchange.
Was generative AI tooling used to co-author this PR?
{pr_number}.significant.rst, in airflow-core/newsfragments. You can add this file in a follow-up commit after the PR is created so you know the PR number.