thesuperzapper commented on pull request #8777: URL: https://github.com/apache/airflow/pull/8777#issuecomment-639358255
@ashb, yes, thats fine, I was saying that we would need to generate the `index.yaml`, not that it _had_ to be at the root of a domain. Separately, I think the community needs to agree on what sort of helm chart we want before we merge anything like this PR. At very least, __all Astronomer specific IP needs to be removed__, for example the Docker images. I think we should instead use the helm chart from [`stable/airflow`](https://github.com/helm/charts/tree/master/stable/airflow), making a few changes, because. that chart has more features than this one, and is not affiliated with any companies. As the official helm chart repo is [going away](https://github.com/helm/charts#deprecation-timeline), the maintainers of `stable/airflow` need to find a new home for it anyway, so we would be more than happy for it to live in an offical airflow repo, whether that is `apache/airflow` or a new `apache/airflow-helm`. ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected]
