HomePhabricator

rm DagsterKubernetesPodOperator and test infra around it

Description

rm DagsterKubernetesPodOperator and test infra around it

Summary:
Why we are deleting this, as per nate:
a) Its based on vendoring an older version of Airflow, so we’re pinned to that version which has known security vulnerabilities and it does not work with newer versions, so we’ll need to rewrite this anyway for newer Airflow if/when we need it for rea
b) I don’t think we have any users on it;
c) I doubt test coverage is very good, since it involves integration testing w/ both Airflow + K8s
d) it is currently broken in master unless you configure the container that it calls in a strange way

Test Plan: BK

Reviewers: nate, johann, alangenfeld

Reviewed By: alangenfeld

Differential Revision: https://dagster.phacility.com/D4814

Details

Provenance
dgibsonAuthored on Oct 19 2020, 7:48 PM
Reviewer
alangenfeld
Differential Revision
D4814: rm DagsterKubernetesPodOperator and test infra around it
Parents
R1:7aa975ca24e4: Change the default system for loading and executing user code to be gRPC…
Branches
Unknown
Tags
Unknown