kedro-org / kedro-plugins

First-party plugins maintained by the Kedro team.
Apache License 2.0
91 stars 83 forks source link

Update airflow documentation for latest version of airflow #573

Closed stichbury closed 5 months ago

stichbury commented 1 year ago

Another task that comes out of https://github.com/kedro-org/kedro/issues/2435

It would be sensible to run through the docs that we are responsible for to confirm they are still OK for the latest version, and meanwhile to caution the reader as to the limitations of version support as noted above.

stichbury commented 1 year ago

I'm not totally sure what version of Airflow the kedro-airflow plugin is tested with/documented against.

The test_requirements.txt file says apache-airflow < 3.0, but the latest version is 2.6, so 🤷

It would be helpful to know for each of the deployment docs (for future review/update) when they were last tested and against what. I think this should be a standard for every touchpoint in the docs going forward.

stichbury commented 1 year ago

Hi @sbrugman I wondered if you have any thoughts on the current state of the Kedro docs about Airflow as @astrojuanlu advised me that you've been working on PRs for kedro-airflow recently, so are presumably well-qualified to judge the current status/quality of the docs.

  1. Are they working at all, or sadly out of date?
  2. If you had to estimate the amount of work needed to update them to the latest version, as a T-shirt size, would it be Small, Medium, Large or other?
sbrugman commented 1 year ago

At this moment the kedro-airflow-k8s plugin is not maintained. Users can use the official kedro-airflow plugin, with a custom template for users on kubernetes. (There is no need for two plugins in practice)

merelcht commented 5 months ago

Closing in favour of https://github.com/kedro-org/kedro-plugins/issues/605