shohamy7 commented on issue #35353:
URL: https://github.com/apache/airflow/issues/35353#issuecomment-1856750083

   Hi! @ahipp13 
   I saw your issue, and took a look in the current helm chart (1.12.0-dev) and 
it looks that the first proposal of supporting extra configuration such as 
`sizeLimit` when using `emptyDir` is already implemented, so I guess it will be 
available in the next release.
   Regard to your second proposal, I agree with @jscheffl that we should not 
add liveness and readiness probe "just to add" them. As @jscheffl said, it 
seems like those components don't have probes for a reason.
   It feels odd to add "dummy" probes for no reason and can make the chart more 
complex with no actual benefit.


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