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

fix: respect connection ID and impersonation in GKEStartPodOperator #36861

Merged
merged 1 commit into from
Jan 20, 2024

Conversation

m1racoli
Copy link
Contributor

The GKEStartPodOperator accepts gcp_conn_id and impersonation_chain as parameters.

This PR ensures that those values are passed to and supported by the corresponding hooks and triggers in deferrable and non-deferrable mode.


^ 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.rst or {issue_number}.significant.rst, in newsfragments.

@boring-cyborg boring-cyborg bot added area:providers provider:cncf-kubernetes Kubernetes provider related issues provider:google Google (including GCP) related issues labels Jan 18, 2024
The GKEStartPodOperator accepts `gcp_conn_id` and `impersonation_chain`
as parameters.

This PR ensures that those values are passed to and supported by the corresponding
hooks and triggers in deferrable and non-deferrable mode.
@m1racoli m1racoli force-pushed the gke-start-pod-impersonation branch from 133caaa to 03c125c Compare January 18, 2024 09:09
@m1racoli m1racoli marked this pull request as ready for review January 18, 2024 09:15
@eladkal eladkal merged commit f1758fd into apache:main Jan 20, 2024
54 checks passed
@m1racoli m1racoli deleted the gke-start-pod-impersonation branch January 22, 2024 16:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:providers provider:cncf-kubernetes Kubernetes provider related issues provider:google Google (including GCP) related issues
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants