Miretpl commented on issue #69571: URL: https://github.com/apache/airflow/issues/69571#issuecomment-4929644837
Thanks for creating this issue! > Also, I made the post sleep command 20 becuase there is no TerminationGracePeriod defined for the pgbouncer deployment in the chart, so it ends up defaulting to 30s. Basically, the sleep 120 doesn't really do what it says and is therefore misleading to users of the chart. IMO, sleep after is pretty much useless; if it is shorter (total command exec time) than the grace period, it doesn't have any effect; if it is longer, it will make the grace period longer by 2 sec, so basically it could be deleted (unless I'm missing something). -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
