Skip to content

Conversation

@ephraimbuddy
Copy link
Contributor

When providers are installed from sources, the python_kubernetes_script.jinja2 is missing in the kubernetes provider package.

This commit adds the file to package_data so it's available in the kubernetes provider package when installed from sources

When providers are installed from sources, the `python_kubernetes_script.jinja2`
is missing in the kubernetes provider package.

This commit adds the file to package_data so it's available in the kubernetes
provider package when installed from sources
@ephraimbuddy ephraimbuddy requested a review from potiuk November 14, 2023 17:38
@potiuk
Copy link
Member

potiuk commented Nov 14, 2023

Interesting. I am working on converting airflow installation of packages to more modern ways (next steps after #35586 and #35617) where the setup.py and those weird behaviours won't be needed any more, but for now it's good to merge that one.

@ephraimbuddy ephraimbuddy merged commit 9ae57d0 into apache:main Nov 14, 2023
@ephraimbuddy ephraimbuddy deleted the add-k8s-script-template branch November 14, 2023 20:10
@ephraimbuddy ephraimbuddy added the changelog:skip Changes that should be skipped from the changelog (CI, tests, etc..) label Nov 20, 2023
@ephraimbuddy ephraimbuddy added this to the Airflow 2.8.0 milestone Nov 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog:skip Changes that should be skipped from the changelog (CI, tests, etc..)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants