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

helm pull stable/airflow pulls the [stable/airflow] chart from github.com/helm/charts repository and not this repository. #10486

Closed
axk7812 opened this issue Aug 22, 2020 · 6 comments
Labels
area:helm-chart Airflow Helm Chart kind:bug This is a clearly a bug kind:documentation

Comments

@axk7812
Copy link

axk7812 commented Aug 22, 2020

Apache Airflow version:
1.10.10

Kubernetes version (if you are using kubernetes) (use kubectl version):
1.17.0

Environment:
AWS
Linux x86_64 x86_64 x86_64 GNU/Linux
Helm 3.2.0

What happened:
Following the directions to install the chart in this repository, and instead it pulls the chart from https://github.com/helm/charts/tree/master/stable/airflow

What you expected to happen:
The chart from https://github.com/apache/airflow/tree/master/chart should be pulled.

You can verify that the values.yaml pulled matches the one from helm/stable rather than the one in apache/airflow chart.

How to reproduce it:

kubectl create namespace airflow
helm repo add stable https://kubernetes-charts.storage.googleapis.com
helm dep update
helm install airflow . --namespace airflow

As minimally and precisely as possible. Keep in mind we do not have access to your cluster or dags.

If you are using kubernetes, please attempt to recreate the issue using minikube or kind.

If this is a UI bug, please provide a screenshot of the bug or a link to a youtube video of the bug in action

How often does this problem occur? Once? Every time etc?

@axk7812 axk7812 added the kind:bug This is a clearly a bug label Aug 22, 2020
@boring-cyborg
Copy link

boring-cyborg bot commented Aug 22, 2020

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

@mik-laj
Copy link
Member

mik-laj commented Aug 22, 2020

Our Helm Chart has been released yet. To install it you need to download source and install this Chart from it.

@potiuk
Copy link
Member

potiuk commented Aug 23, 2020

@dimberman -> is this something we should close as invalid or maybe we should already have some project (similar to the Docker image project https://github.com/apache/airflow/projects/3 where we could gather state and what's left to complete helm chart productionising. I am happy to put similar issues there while scanning, but I believe there is more work to be done to make the helm chart ready for production? WDYT? Is this is something you plan to do ?

@akuzni2
Copy link

akuzni2 commented Aug 23, 2020

@mik-laj - thanks!
@potiuk - not sure if this is invalid. The documentation on the chart page https://github.com/apache/airflow/tree/master/chart shows to install the chart from stable helm repo. Shouldn't we just update the documentation for it to reflect that it should be installed from source like @mik-laj mentioned?

@potiuk
Copy link
Member

potiuk commented Aug 23, 2020

I'd leave it up to @dimberman

@mik-laj mik-laj added the area:helm-chart Airflow Helm Chart label Oct 9, 2020
@mik-laj
Copy link
Member

mik-laj commented Mar 6, 2021

Now the documentation no longer references the wrong repository. We also have a warning at the beginning of the README.md file that this Chart has not been released.
Screenshot 2021-03-07 at 00 30 36
I don't think we can do anything else but release a Helm Chart, which is described in another ticket. See: #10752

Related: #14643

@mik-laj mik-laj closed this as completed Mar 6, 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:bug This is a clearly a bug kind:documentation
Projects
None yet
Development

No branches or pull requests

4 participants