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
Details
Details
BK
Diff Detail
Diff Detail
- Repository
- R1 dagster
- Branch
- killtheoperator (branched from master)
- Lint
Lint OK - Unit
No Unit Test Coverage
Event Timeline
Comment Actions
As per discussion in the platform lane chat.
worth updating this with the relevant contents of - since this is a public artifact and it may be useful for someone to see why we let this go
Comment Actions
include proactive CHANGES.md entry for this (just add (Upcoming) to the new version # section)
