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

start peering connection to neighbors from node's advertise-ip #777

Merged
merged 1 commit into from
Mar 11, 2022

Conversation

tamihiro
Copy link
Contributor

@tamihiro tamihiro commented Sep 5, 2019

This patch explicitly sets local-address with node's advertise-ip for each neighbor configuration.

@aauren
Copy link
Collaborator

aauren commented Jul 10, 2020

@tamihiro Have you tested these changes in your cluster and found that they fix #778 ?

@murali-reddy The idea behind this work seems like it might be a reasonable fit for 1.1, do you have any thoughts on this?

@aauren aauren requested a review from murali-reddy July 10, 2020 07:14
@tamihiro
Copy link
Contributor Author

@tamihiro Have you tested these changes in your cluster and found that they fix #778 ?

Yes this fixes #778.

kube-routers in my cluster need this and also #779 to work properly.
https://github.com/tamihiro/kube-router/commits/mydev02

@aauren
Copy link
Collaborator

aauren commented Apr 11, 2021

@tamihiro Sorry, that this is taking so long. Recently, the project has had other priorities.

RE: #778 (comment) I'm seeking Murali's thoughts. You can wait for that to go through, or if you're willing it would help if you would proactively rebase so this is ready whenever I hear back.

@tamihiro
Copy link
Contributor Author

@aauren Sorry for taking it so long, I did rebase the current master.

@aauren aauren merged commit 184976a into cloudnativelabs:master Mar 11, 2022
@aauren
Copy link
Collaborator

aauren commented Mar 11, 2022

@tamihiro Thanks for doing all of this work. Sorry it took so long for us to finally come back around to this!

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