Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

vrrp: handle a reload with no more startup_delay #2559

Merged
merged 1 commit into from
Mar 28, 2025

Conversation

louis-6wind
Copy link
Contributor

During the vrrp_startup_delay time, if keepalived is reloaded with no more startup_delay, the startup_delay is never timed out and all received adverts is discarded.

The commit causes the startup_delay timer to be reinstated after a reload with no more startup_delay if the timer has not yet expired.

Fixes: 58483b2 ("vrrp: Handle a reload before vrrp_delayed_start has expired")

During the vrrp_startup_delay time, if keepalived is reloaded with
no more startup_delay, the startup_delay is never timed out and all
received adverts is discarded.

The commit causes the startup_delay timer to be reinstated after a
reload with no more startup_delay if the timer has not yet expired.

Fixes: 58483b2 ("vrrp: Handle a reload before vrrp_delayed_start has expired")
Signed-off-by: Louis Scalbert <[email protected]>
@louis-6wind
Copy link
Contributor Author

Please review this

@pqarmitage pqarmitage merged commit 6049a4e into acassen:master Mar 28, 2025
10 checks passed
@louis-6wind
Copy link
Contributor Author

Thank you for the merge @pqarmitage

I will address your remark on the other pull-request soon.
The work on fault-init-exit-delay is not finished and we will let you know when it is finished

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants