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

Add strict ARP enabling note #507

Merged
merged 1 commit into from Mar 2, 2020
Merged

Add strict ARP enabling note #507

merged 1 commit into from Mar 2, 2020

Conversation

kvaps
Copy link
Contributor

@kvaps kvaps commented Dec 16, 2019

This PR describes #153 (comment) in installation steps

@kvaps kvaps changed the title Add strictArp enabling note Add strict ARP enabling note Dec 16, 2019
@kvaps kvaps changed the title Add strict ARP enabling note strict ARP enabling note Dec 16, 2019
@kvaps kvaps changed the title strict ARP enabling note Add strict ARP enabling note Dec 16, 2019
@kvaps kvaps force-pushed the ipvs-warning branch 2 times, most recently from b4cb597 to 8f7384c Compare Dec 16, 2019
@champtar
Copy link
Contributor

champtar commented Feb 28, 2020

@kvaps could you rebase ?

@kvaps kvaps force-pushed the ipvs-warning branch 2 times, most recently from 902f8bf to c7862a5 Compare Mar 2, 2020
@kvaps
Copy link
Contributor Author

kvaps commented Mar 2, 2020

sure, rebased

@champtar
Copy link
Contributor

champtar commented Mar 2, 2020

@daxmc99 @danderson can one of you merge this one ?

daxmc99
daxmc99 approved these changes Mar 2, 2020
Copy link
Contributor

@daxmc99 daxmc99 left a comment

I don't have a tone of experience with ipvs but this note makes sense

@danderson danderson merged commit 6572097 into metallb:main Mar 2, 2020
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.

None yet

4 participants