Configurable cron schedule for reconciler

k8snetworkplumbingwg / whereabouts

A CNI IPAM plugin that assigns IP addresses cluster-wide

Apache License 2.0

273 stars 120 forks source link

Configurable cron schedule for reconciler #398

Closed maiqueb closed 6 months ago

maiqueb commented 6 months ago

What this PR does / why we need it: This PR builds on top of #392 (which adds a config map from where the reconciler scheduler can be configured via a config map) to react to updates of said configuration.

Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged): Fixes #

Special notes for your reviewer (optional): Tested locally.

maiqueb commented 6 months ago

@dougbtv I've added a unit test that shows this working.

I think that's enough ...

I'm OK with merging. Let's talk about it in 2024.