Skip to content

Conversation

@cgoncalves
Copy link
Contributor

No description provided.

net.ParseIP() returns 16-byte IPv4-mapped IPv6 format where IPv4 bytes
are at the END, not beginning. [4]byte(garp.IP) took wrong bytes.

Fixed by calling To4() before, forcing validated creation via
NewGARP(). Interface prevents bypassing extracting the correct IPv4 address.

Signed-off-by: Patryk Diak <[email protected]>
IPv6 equivalent of GARP functionality for EgressIP failover. Sends
unsolicited NAs when IPv6 addresses are added to secondary host interfaces.

Signed-off-by: Patryk Diak <[email protected]>
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Oct 29, 2025

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

@openshift-ci openshift-ci bot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Oct 29, 2025
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Oct 29, 2025

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: cgoncalves
Once this PR has been reviewed and has the lgtm label, please assign martinkennelly for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants