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

OS-8196 Native zone support for link-local routes #281

Merged
merged 2 commits into from
Jul 9, 2020
Merged

OS-8196 Native zone support for link-local routes #281

merged 2 commits into from
Jul 9, 2020

Conversation

hww3
Copy link

@hww3 hww3 commented Apr 1, 2020

Hello-

@mgerdts did an initial review of this change last year, and I have have been running this change on a number of SmartOS systems since that time. Details of the situation this aims to correct are described in #244.

I've specifically verified the following scenarios:

  • no static routes (no errors)
  • link-local routes in combination with a default route (no errors and link local route created first)
  • ordering of non-link-local routes in combination with link-local routes (no errors and link-local route created before non-link-local route)

@jasonbking
Copy link

Everything there seems reasonable -- can you provide any testing notes?

@hww3
Copy link
Author

hww3 commented Jul 9, 2020

Hi @jasonbking... of course, here's what I did to verify:

I have a few nodes at OVH that use "failover" ip addresses. These addresses are handed out on a CIDR subnet different than the admin network. The default route, however, is that of the default network. So a link-local route needs to be created before the default route. That's the rationale for this change.

I've been using this patch on 10-15 native zones across multiple hosts for the past year. These zones require the link local route to be created first as the default route is on the link-local network (that isn't the same as the IP address of the interface.) Without the patch, creating the default route fails with a "no route to host" error, but is successful with the patch.

I've also been running platform images with this fix on multiple systems that don't have static routes at all, and these all work as expected with the the patch.

I created a few (3 or 4) zones with a static (but not default) route in a lab setup. These all seemed to work just fine, in that the static routes were added as expected in the "non link-local" path.

I'm happy to provide any additional details you think would be helpful. Just let me know!

@jasonbking jasonbking changed the title joyent/illumos-joyent#244 Native zone support for link-local routes OS-8196 Native zone support for link-local routes Jul 9, 2020
@danmcd
Copy link

danmcd commented Jul 9, 2020

Thank you for this.

@danmcd
Copy link

danmcd commented Jul 9, 2020

@hww3 -- I've noticed, BTW, that the normal /etc/inet/static_routes path in upstream illumos MIGHT benefit from some of this work. Care to take a crack at it in illumos-gate? I can help with integrating-to-gate's idiosyncracies.

@jasonbking jasonbking merged commit 34dbf83 into TritonDataCenter:master Jul 9, 2020
@hww3
Copy link
Author

hww3 commented Jul 10, 2020

Hi @danmcd-

I suspect you're right... I'll take a look and see if that part can be improved as well. I'll get in touch as I have questions about integrating!

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

Successfully merging this pull request may close these issues.

None yet

3 participants