You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Currently it can happen that there are multiple events scheduled from reSchedule funtionality. This should not happen, there should be at most one, possibly the earliest keeped in such case. In addition to that we have a automated retry functionality, so we have to think about how this two should work together.
Ideally there is always only 1 of these to is scheduled types scheduled. But also in general there should be at most one timed event coming either of these functionalities.
The text was updated successfully, but these errors were encountered:
The rationale for this is that if you schedule a reconciliation to happen at some later time in the future it's because you don't expect a reconciliation to happen "naturally" because of a cluster-triggered event. It follows, then, that if a reconciliation is triggered before the specified time delay, that scheduled reconciliation should be cancelled.
Similarly, if a reconciliation is scheduled at some future time and a new request for rescheduling comes in, then only the rescheduling request that would trigger an earlier reconciliation should be kept (for the same reason: a rescheduling request is meant to ensure at least one reconciliation happens by the requested delay, so if a reconciliation happens before, then the request should be considered fulfilled).
every successful reconciliation run will cancel the scheduled events (either re-schedule or retry)
if a retry or re-schedule is happening, it will also cancel the previous schedules.
@metacosm I think we should take just the "last instruction" wins. Not taking into account the timing. So even if a new re-schedule happens but there is already one with an earlier execution planned, the new one should be the lead and we should cancel the old one. This makes sense because the new one is set by a more recent reconciliation aware of the current state.
Uh oh!
There was an error while loading. Please reload this page.
Currently it can happen that there are multiple events scheduled from
reSchedule
funtionality. This should not happen, there should be at most one, possibly the earliest keeped in such case. In addition to that we have a automated retry functionality, so we have to think about how this two should work together.Ideally there is always only 1 of these to is scheduled types scheduled. But also in general there should be at most one timed event coming either of these functionalities.
The text was updated successfully, but these errors were encountered: