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

Tweak kernel ARP settings for keepalived to work. #137

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

meisterT
Copy link
Member

@meisterT meisterT commented May 5, 2024

This has not been necessary in the past, but was required at ICPC WFs in Egypt, see also:
https://keepalived.readthedocs.io/en/latest/software_design.html#note-on-using-vrrp-with-virtual-mac-address

@meisterT
Copy link
Member Author

meisterT commented May 5, 2024

cc @eldering

Not sure whether we want this in general

Copy link
Member

@vmcj vmcj left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This now depends on the CLUSTER_ID which is something we probably don't want to keep.

@nickygerritsen
Copy link
Member

This now depends on the CLUSTER_ID which is something we probably don't want to keep.

We should add a conditional for this I guess, since we only need these three when we have a cluster. But then again, we only have keepalived when we have a cluster right?

@vmcj
Copy link
Member

vmcj commented May 7, 2024

This now depends on the CLUSTER_ID which is something we probably don't want to keep.

We should add a conditional for this I guess, since we only need these three when we have a cluster. But then again, we only have keepalived when we have a cluster right?

No, CLUSTER_ID is for the keepalived cluster, in a hypothetical case you have a network blue with 4 domservers where 2 replicate with each other (let's call them wf46 & wf47) than they would use the same mac in both clusters and you get 50% of the packets because the same mac exists. So its only related to how many distinct keepalived clusters you run on the same network. So it could be useful when pc2 also does keepalived for example.

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.

4 participants