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

Posted to commits@airflow.apache.org by "N-ickJones (via GitHub)" <gi...@apache.org> on 2023/02/28 21:26:22 UTC

[GitHub] [airflow] N-ickJones commented on pull request #29818: updates values.yaml config.kubernetes to config.kubernetes_executor

N-ickJones commented on PR #29818:
URL: https://github.com/apache/airflow/pull/29818#issuecomment-1448938010

   > Thanks for the PR @N-ickJones!
   > 
   > Unfortunately, it's not quite as simple as your proposed change. The latest version of the chart still supports older Airflow versions. Check out what was done for the [logging config move](https://github.com/apache/airflow/blob/4b36137a31241d0f502604213546b6bf677fea69/chart/values.yaml#L1784-L1789).
   
   For backwards compatibility do my updated changes work. Similar to the logging config move, add a duplicate section.


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