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

Release Helm Chart officially #10752

Closed
potiuk opened this issue Sep 5, 2020 · 16 comments
Closed

Release Helm Chart officially #10752

potiuk opened this issue Sep 5, 2020 · 16 comments
Assignees
Labels
area:helm-chart Airflow Helm Chart kind:feature Feature Requests

Comments

@potiuk
Copy link
Member

potiuk commented Sep 5, 2020

Description

When we are ready, we should release the Helm Chart officially.

Use case / motivation

Discussion in Apache Software Foundation started: helm/charts#23685 about standardising policies and tools for releasing Helm Charts in ASF and we should follow it.

@potiuk
Copy link
Member Author

potiuk commented Sep 5, 2020

CC: @dimberman @kaxil @ashb

@potiuk potiuk added the area:helm-chart Airflow Helm Chart label Sep 5, 2020
@mik-laj
Copy link
Member

mik-laj commented Oct 23, 2020

I would like to leave a note here that if we publish the documentation on ArtifactsHub, we will have user-friendly reference documentation.
See:
#11077

@dimberman
Copy link
Contributor

@mik-laj That's fantastic thank you!!

@ryw ryw added the dont-merge label Oct 29, 2020
@ryw
Copy link
Member

ryw commented Oct 29, 2020

dropping a note here that we decided to defer this to post Airflow 2.0 https://cwiki.apache.org/confluence/display/AIRFLOW/Airflow+2.0+-+Planning

@ryw ryw removed the dont-merge label Oct 29, 2020
@dnskr
Copy link
Contributor

dnskr commented Jan 19, 2021

Hi @potiuk, @ryw and others!
Do we have any updates on this issue?
Can I use this chart or can I help somehow to move forward?

I am asking because current airflow helm chart from https://charts.helm.sh/stable is still community edition which is not relevant to the doc

$ helm repo list
NAME                    URL
stable                  https://charts.helm.sh/stable

$ helm search repo stable/airflow
NAME            CHART VERSION   APP VERSION     DESCRIPTION
stable/airflow  7.13.3          1.10.12         DEPRECATED - please use: https://github.com/air...

@potiuk
Copy link
Member Author

potiuk commented Jan 19, 2021

I hope we can discuss the steps to make helm "released" at the next "dev" meeting which is going to be tomorrow ..

@dnskr
Copy link
Contributor

dnskr commented Jan 19, 2021

I'll be happy to contribute into it if needed.

@kaxil
Copy link
Member

kaxil commented Feb 11, 2021

Next Steps:

I will start creating issues for each point.

Added Notes for now: https://github.com/apache/airflow/projects/7?fullscreen=true

@kaxil
Copy link
Member

kaxil commented Feb 11, 2021

@dnskr Can you start looking at tests and add where they are missing, that would be a great help

@thesuperzapper
Copy link
Contributor

Just my 2c, but I also think there is a significant amount of documentation lacking which will make a bad first experience with the chart:

  • the values.yaml file is missing comments for most values
  • more general explanations are pretty sparse (it seems we currently only have short paragraphs in the README)
  • examples for common environments are missing (GKE, EKS, On-Prem)

Related to this (and the discussion about versioning strategy), we should consider:

  • publishing the doc's seperate to airflow itself, (because I assume we will release the chart at a different cadence to airflow)
  • provide upgrade guides for major/minor version bumps of the chart

@kaxil
Copy link
Member

kaxil commented Feb 11, 2021

Good points @thesuperzapper - I added a point to review and add lacking documentation (we can expand on the details after we create issues for each of those points).

@kaxil
Copy link
Member

kaxil commented Feb 18, 2021

The following Project Board should help with tracking the progress as well as to all of you who would like to help with the process.

https://github.com/apache/airflow/projects/7?fullscreen=true

@lewismc
Copy link
Member

lewismc commented Feb 19, 2021

This would be super helpful folks. I'll try my best to provide more PR's on the Chart as my experiments evolve and knowledge improves.

@kaxil
Copy link
Member

kaxil commented May 20, 2021

The first version of the official Helm Chart was finally released yesterday.

Mailing List Announcement: https://lists.apache.org/thread.html/r131d839158b8a7a92a7813183cae30d248be9e330ea2faaf9e654970%40%3Cdev.airflow.apache.org%3E 🎉
📦 ArtifactHub: https://artifacthub.io/packages/helm/apache-airflow/airflow
📚 Docs: ⁨https://airflow.apache.org/docs/helm-chart/1.0.0/
🚀 Quick Start Installation Guide: https://airflow.apache.org/docs/helm-chart/1.0.0/quick-start.html

Thanks to all the contributors who made this possible.

@kaxil kaxil closed this as completed May 20, 2021
@potiuk
Copy link
Member Author

potiuk commented May 20, 2021

🎉 🎉 🎉 🎉 🎉 🎉 🎉 🎉 🎉 🎉 🎉 🎉 🎉 🎉

@kaxil kaxil self-assigned this May 20, 2021
@lewismc
Copy link
Member

lewismc commented May 20, 2021

Real nice work 🎉

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

8 participants