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

Set MTU according to DHCP provided option #113

Closed
wants to merge 1 commit into from

Conversation

fwiesel
Copy link

@fwiesel fwiesel commented Feb 28, 2024

Fixes #112

@smoser
Copy link
Member

smoser commented Feb 28, 2024

This looks good, thank you for doing that. Could you please mention Fixes #112 in the commit message?

Last, can you just test the images / verify that this actually fixes your issue (c-i will publish images in the actions) and then I'd like to land.

Thanks.

@fwiesel
Copy link
Author

fwiesel commented Mar 6, 2024

I have trouble verifying that the patch works with the image from the pipeline as it is missing drivers for my environment (Vsphere).
I can confirm it works with those modules included.

@fwiesel fwiesel closed this Aug 1, 2024
@fwiesel fwiesel deleted the mtu branch August 1, 2024 15:57
@fwiesel fwiesel restored the mtu branch August 1, 2024 16:00
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.

Cirros image does not configure MTU according to DHCP
2 participants