potiuk commented on a change in pull request #17743:
URL: https://github.com/apache/airflow/pull/17743#discussion_r693233524



##########
File path: chart/templates/triggerer/triggerer-deployment.yaml
##########
@@ -0,0 +1,212 @@
+# Licensed to the Apache Software Foundation (ASF) under one

Review comment:
       Same as in docker compose, I believe this won't work for Airflow pre 
2.2. But people will use Helm Chart for 2.1 and 2.0 versions of airflow for 
quite a while. Should we make some ifs here?




-- 
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-unsubscr...@airflow.apache.org

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


Reply via email to