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

VSC static route nexthop is wrong #575

Open
jbemmel opened this issue Mar 10, 2018 · 4 comments
Open

VSC static route nexthop is wrong #575

jbemmel opened this issue Mar 10, 2018 · 4 comments

Comments

@jbemmel
Copy link
Collaborator

jbemmel commented Mar 10, 2018

https://github.com/nuagenetworks/nuage-metro/blob/master/roles/vsc-deploy/templates/config.cfg.j2

Next hop should be control IP of each peer, if in same subnet

Instead of /32 static routes, it's better to peer on control IP and add a default route only

@ghost
Copy link

ghost commented Mar 11, 2018

The existing line is:

        static-route {{ hostvars[vsc_item].system_ip }}/32 next-hop {{ vsc_control_ip }}

I think you are suggesting that we drop this line and, instead, add do two things?

  1. Peer on control IP (How do we do this?)
  2. Create a default route (How do we do this?)

@jbemmel
Copy link
Collaborator Author

jbemmel commented Mar 11, 2018 via email

@ghost
Copy link

ghost commented Apr 20, 2018

@jbemmel I'm trying to understand how to fix this issue. In one place, you wrote that the next hop should be the control ip of each peer. In another, you said it should be the control_network_gw. We do not collect the control_network_gw in build vars. What exactly is the fix here?

@jbemmel
Copy link
Collaborator Author

jbemmel commented Apr 20, 2018

The way I would configure it, is a single static default route to the control network gateway if available- if the vscs are all in a single subnet and you are not connecting to dcgws, then you could also leave the static route out. The control interface ips should be used as bgp neighbors, not as the static route next hop

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

No branches or pull requests

1 participant