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

Use kubeconfig as one of the optional parameters for creating KubernetesHook #39227

Open
2 tasks done
luoyuliuyin opened this issue Apr 24, 2024 · 1 comment · May be fixed by #39249
Open
2 tasks done

Use kubeconfig as one of the optional parameters for creating KubernetesHook #39227

luoyuliuyin opened this issue Apr 24, 2024 · 1 comment · May be fixed by #39249
Labels
area:providers kind:feature Feature Requests provider:cncf-kubernetes Kubernetes provider related issues

Comments

@luoyuliuyin
Copy link
Contributor

luoyuliuyin commented Apr 24, 2024

Description

When we use KubernetesPodOperator, we need to create KubernetesHook in order to connect to Kubernetes services. There are currently 3 supported methods for creating KubernetesHook:
1、environment variable method,
2、kubeconfig_path method,
3、db connection variable method.
image

However, these methods are all from the perspective of the airflow system owner. The airflow owner can change the variables of the airflow_worker, create and modify files in the airflow_worker, and operate the airflow_db. However, in many cases, the users of airflow are not The owner of airflow, The user does not have the authority to make changes to the airflow_worker, nor does it have read and write permissions to the db, nor should it see the data stored in the db by other users.

Therefore, it is best for users to manage their own data rather than hosting it on the airflow system, in this case, it is a relatively reasonable choice to add an optional parameter to receive the kubeconfig text.

Use case/motivation

add an optional parameter to receive the kubeconfig text
image

If this proposal is accepted, I will happily submit a PR to support this feature

Related issues

No response

Are you willing to submit a PR?

  • Yes I am willing to submit a PR!

Code of Conduct

@luoyuliuyin luoyuliuyin added kind:feature Feature Requests needs-triage label for new issues that we didn't triage yet labels Apr 24, 2024
@luoyuliuyin
Copy link
Contributor Author

I submitted a PR to support this feature, #39249

@nathadfield nathadfield added area:providers provider:cncf-kubernetes Kubernetes provider related issues and removed needs-triage label for new issues that we didn't triage yet labels Apr 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:providers kind:feature Feature Requests provider:cncf-kubernetes Kubernetes provider related issues
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants