David de L. created AIRFLOW-5873:
------------------------------------

             Summary: KubernetesPodOperator fixes and test
                 Key: AIRFLOW-5873
                 URL: https://issues.apache.org/jira/browse/AIRFLOW-5873
             Project: Apache Airflow
          Issue Type: Bug
          Components: operators
    Affects Versions: 1.10.6
            Reporter: David de L.
            Assignee: David de L.
             Fix For: 1.10.7


* `KubernetesPodOperator` kwarg `resources` is erroneously passed to 
`base_operator`, instead should only go to `PodGenerator`. The two have 
different syntax. (both on `master` and `v1-10-test` branches)
 * `KubernetesPodOperator` kwarg `in_cluster` erroneously defaults to False in 
comparison to `default_args.py`
 * `kubernetes/pod.py`: `Resources` does not have `__slots__` so accepts 
arbitrary values in `setattr` (present on `v1-10-test` branch 
[https://github.com/apache/airflow/blame/v1-10-test/airflow/contrib/operators/kubernetes_pod_operator.py#L166])
 * `v1-10-test` is behind `master` with KubernetesPodOperator fixes and 
refactors
 ** e.g. move kubernetes folder one level up from `/contrib` 
[https://github.com/apache/airflow/blame/4dd24a2c595d4042ffe745aed947eaaea6abb652/airflow/contrib/operators/kubernetes_pod_operator.py#L21]
 ** fix `xcom_push` to `do_xcom_push` 
[https://github.com/apache/airflow/blame/4dd24a2c595d4042ffe745aed947eaaea6abb652/airflow/contrib/operators/kubernetes_pod_operator.py#L90]



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to