You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by "Ryan Williams (JIRA)" <ji...@apache.org> on 2019/06/16 23:41:00 UTC
[jira] [Created] (AIRFLOW-4800) GKE operators call GKEClusterHook
with wrong (old) arguments
Ryan Williams created AIRFLOW-4800:
--------------------------------------
Summary: GKE operators call GKEClusterHook with wrong (old) arguments
Key: AIRFLOW-4800
URL: https://issues.apache.org/jira/browse/AIRFLOW-4800
Project: Apache Airflow
Issue Type: Bug
Components: contrib
Affects Versions: 1.10.3
Reporter: Ryan Williams
Assignee: Ryan Williams
[#4364|https://github.com/apache/airflow/pull/4364/files#diff-6c81e7d223790753a57708ed860b51d1R39] changed GKEClusterHook's ctor arguments, but not call-sites in [GKEClusterDeleteOperator|https://github.com/apache/airflow/blob/2d18f0738f775183b06d19f0b6703285c8bdcb32/airflow/contrib/operators/gcp_container_operator.py#L89] and [GKEClusterCreateOperator|https://github.com/apache/airflow/blob/2d18f0738f775183b06d19f0b6703285c8bdcb32/airflow/contrib/operators/gcp_container_operator.py#L176].
[Relevant tests|https://github.com/apache/airflow/blob/2d18f0738f775183b06d19f0b6703285c8bdcb32/tests/contrib/operators/test_gcp_container_operator.py#L57-L58] still assert that the old arguments are passed.
I'd have thought that the {{location}} arg being passed to {{delegate_to}} would cause real usages of the GKE operators to break, but perhaps it doesn't and they are working. I'm not sure what real-world use of these operators is like.
I think the fix is trivial and will PR it shortly.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)