Skip to content

Handle updated vrrp_delayed_start and return (none) from inet_sockaddrtos() if address unspecified #2580

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

Merged
merged 6 commits into from
Mar 28, 2025

Conversation

pqarmitage
Copy link
Collaborator

No description provided.

Previously if the address was unspecified the formatted address
from the previous call was returned, rather than something
indicating that there was no address.

Signed-off-by: Quentin Armitage <[email protected]>
We can't do anything with them, and they are not usable, so we now
wait until we are notified that the address is no longer tentative
before we consider using it.

Signed-off-by: Quentin Armitage <[email protected]>
If an existing base interface of a VMAC is renamed, delete the VMAC
since the configure base interface no longer exists.

When an existing interface is renamed to match the base interface for
a VMAC for a VRRP instance, for IPv6 when the VMAC interface is created
a link local address is added, so clear the NO_ADDRESS fault flag by
calling try_up_instance(). For IPv4 we do not add an address to the VMAC
when it is created, so will wait for notlink notifications of addresses
on the base interface, which can then be added to be VMAC.

Since 4c7a94a ("vrrp: Make VMAC IPv6 link local address mirror parent
interface"), it was not called anymore when a IPv6 link-local was back
on the VMAC interface.

Fixes: 4c7a94a ("vrrp: Make VMAC IPv6 link local address mirror parent interface")

Based on a patch signed-off-by: Louis Scalbert <[email protected]>

Signed-off-by: Quentin Armitage <[email protected]>
keepalived was warning that using ipsets had been specified but
iptables had not been specified, even if ipsets had not been
specified.

Signed-off-by: Quentin Armitage <[email protected]>
The git-commit.h included in a tar file was not the correct version
if keepalived had not been built since the last commit.

Signed-off-by: Quentin Armitage <[email protected]>
@pqarmitage pqarmitage merged commit df377c0 into acassen:master Mar 28, 2025
12 checks passed
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.

1 participant