Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Provide official airflow chart #8270

Closed
eugene-chernyshenko opened this issue Apr 13, 2020 · 8 comments
Closed

Provide official airflow chart #8270

eugene-chernyshenko opened this issue Apr 13, 2020 · 8 comments
Labels
area:helm-chart Airflow Helm Chart kind:feature Feature Requests
Milestone

Comments

@eugene-chernyshenko
Copy link

Could you please provide official chart for your image.

The stable/airflow chart was created for image puckel/docker-airflow.

There are specific features in the puckel/docker-airflow entrypoint you don't have

such as
https://github.com/puckel/docker-airflow/blob/master/script/entrypoint.sh#L13
https://github.com/puckel/docker-airflow/blob/master/script/entrypoint.sh#L57

So your image won't work with stable/airflow correctly

@eugene-chernyshenko eugene-chernyshenko added the kind:feature Feature Requests label Apr 13, 2020
@boring-cyborg
Copy link

boring-cyborg bot commented Apr 13, 2020

Thanks for opening your first issue here! Be sure to follow the issue template!

@rcjsuen
Copy link
Contributor

rcjsuen commented Apr 13, 2020

@eugene-chernyshenko There is an ongoing discussion about this on the [email protected] mailing list.

@potiuk
Copy link
Member

potiuk commented Apr 14, 2020

Indeed. We are working on the helm chart with @dimberman and there already few suggestions on the changes that will need to be added to the image/helm chart, some of them also being discussed for example in this slack discussion: https://apache-airflow.slack.com/archives/CQAMHKWSJ/p1586556315075700

Would you mind describing the use case that you would like to be handled this way?

Some context: just linking to specific lines in Puckel is not enough - the idea is that we want to implement use cases that people want to have - not necessarily in the same way Puckel image/helm chart addressed so far. Rather than copying everything from Puckel/current help chart has, we decided to start from basics and add things as our users will tell us need (but sometimes in a different way that might be easier for community to manage).

So .. How would you explain your use case @eugene-chernyshenko?

@ashb
Copy link
Member

ashb commented Apr 16, 2020

We are actively working on making https://github.com/astronomer/airflow-chart/ suitable for wider use

@jacobhjkim
Copy link
Contributor

Can we close this since https://github.com/apache/airflow/tree/master/chart this was merged?

@potiuk potiuk added this to the Airflow 1.10.11 milestone Jun 26, 2020
@potiuk
Copy link
Member

potiuk commented Jun 26, 2020

I think we have a few things to fix/update and we will close it when we release 1.10.11 with the official chart in

@mik-laj mik-laj added the area:helm-chart Airflow Helm Chart label Oct 9, 2020
@kaxil kaxil modified the milestones: Airflow 1.10.13, Airflow 2.0.0 Oct 30, 2020
@ryw ryw modified the milestones: Airflow 2.0.0 (rc1), Airflow 2.1 Nov 30, 2020
@ryw
Copy link
Member

ryw commented Nov 30, 2020

Moved milestone to 2.1 since we won't release official Helm chart with 2.0.

@kaxil
Copy link
Member

kaxil commented Feb 17, 2021

Duplicate of #10752

@kaxil kaxil marked this as a duplicate of #10752 Feb 17, 2021
@kaxil kaxil closed this as completed Feb 17, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:helm-chart Airflow Helm Chart kind:feature Feature Requests
Projects
None yet
Development

No branches or pull requests

9 participants