danielhoherd opened a new pull request, #32570: URL: https://github.com/apache/airflow/pull/32570
This change is quite simple in that it allows helm templates to be used in the value of `cron.schedule`. This also works with strings, which makes the change backwards compatible, so it will nave no impact on existing installations. Shout out to @jedcunningham and @dstandish for helping me settle on this very simple syntax change. There were no tests for cleanup.schedule, so I added tests, including tests for using strings and for using helm template logic. FWIW, the test logic shows an example of using adler32sum to select a deterministic random-ish number, seeded by `.Release.Name`, to use as the starting minute for the cleanup cronjob. This mechanism will avoids starting all cleanup jobs across all deployments on minute 0, and was the impetus for changing from string to a template. -- 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]
