ashb edited a comment on issue #6623: Donating Google's Airflow K8s Operator to Apache Foundation URL: https://github.com/apache/airflow/pull/6623#issuecomment-556995070 Excited to see we finally got here! A couple of thoughts/comments: - Would this be better off in a different repo to Airflow, say `apache/airflow-kuberentes-controler` - We're in the tricky situation where "Operator" means different things in Kube and Airflow :) - We need licesnse headers in _every_ file :( (The pre-commit hooks set up in this repo should automatically add almost all of them for us, or can be extended to add them for `.go` files if it doesn't already support that. - How much of the architecture is configurable? For example if we want to pre-bake the dags in to an image and don't want the DAG-pv. - Does Kube on AWS (EKS? Too many acronyms) support ReadMany PVs? - Why did you choose Stateful sets for the scheduler and UI, rather than Deployements?
---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services