You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2022/08/03 08:14:43 UTC

[GitHub] [airflow] eff917 commented on pull request #25479: fix helm install command

eff917 commented on PR #25479:
URL: https://github.com/apache/airflow/pull/25479#issuecomment-1203631769

   > I don't think we should document extra config being (seemingly) required for an install to work in the first example on the homepage. If that was the case, it should set it by default anyways.
   > 
   > We should find another place to document this. We almost need a different page for this type of thing, because I also think the ArgoCD section is out of place on the homepage as well.
   
   I agree, but didn't really know where or how to insert it. I see 3 options:
   
   1. this one
   2. put it in a note section under/over the code block
   3. change the subchart/helm chart default value, so this setting is not needed explicitly
   


-- 
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.

To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org