Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add auth_type to LivyHook #25179

Closed
2 tasks done
bdsoha opened this issue Jul 20, 2022 · 2 comments · Fixed by #25183
Closed
2 tasks done

Add auth_type to LivyHook #25179

bdsoha opened this issue Jul 20, 2022 · 2 comments · Fixed by #25183

Comments

@bdsoha
Copy link
Contributor

bdsoha commented Jul 20, 2022

Apache Airflow Provider(s)

apache-livy

Versions of Apache Airflow Providers

apache-airflow-providers-apache-livy==3.0.0

Apache Airflow version

2.3.3 (latest released)

Operating System

Ubuntu 18.04

Deployment

Other 3rd-party Helm chart

Deployment details

No response

What happened

This is a feature request as apposed to an issue.

I want to use the LivyHook to communicate with a Kerberized cluster.
As such, I am using requests_kerberos.HTTPKerberosAuth as the authentication type.
Currently, I am implementing this as follows:

from airflow.providers.apache.livy.hooks.livy import LivyHook as NativeHook
from requests_kerberos import HTTPKerberosAuth as NativeAuth

class HTTPKerberosAuth(NativeAuth):
    def __init__(self, *ignore_args, **kwargs):
        super().__init__(**kwargs)

class LivyHook(NativeHook):
    def __init__(self, auth_type=HTTPKerberosAuth, **kwargs):
        super().__init__(**kwargs)
        self.auth_type = auth_type

What you think should happen instead

No response

How to reproduce

No response

Anything else

No response

Are you willing to submit PR?

  • Yes I am willing to submit a PR!

Code of Conduct

@bdsoha bdsoha added area:providers kind:bug This is a clearly a bug labels Jul 20, 2022
@boring-cyborg
Copy link

boring-cyborg bot commented Jul 20, 2022

Thanks for opening your first issue here! Be sure to follow the issue template!

@eladkal
Copy link
Contributor

eladkal commented Jul 20, 2022

feel free to submit PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants