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

Suggestion: Add ProxyDHCP support #113

Open
lars18th opened this issue Jul 31, 2024 · 3 comments
Open

Suggestion: Add ProxyDHCP support #113

lars18th opened this issue Jul 31, 2024 · 3 comments

Comments

@lars18th
Copy link

Hi,

The use case is very easy (and common): you can't replace the current DHCP server on your network and you can't edit the configuration to add the next-server option to it.

In this case the only option is to use the ProxyDHCP way. How it works? It uses a non-standard booting method where your booting client receives two DHCP OFFER packets. One is from the stock DHCP server with the IP address and all other information. And another one from your booting server with the PXE data. You can read more from here https://github.com/pypxe/PyPXE/blob/master/DOCUMENTATION.md#proxydhcp

So because this project https://github.com/pypxe/PyPXE includes support for this extension, the idea is to include it. I've done some tests with it sudo python -m pypxe.server --dhcp-proxy --tftp and it works with success.

The suggestion is then to add this package to this container and use it to enable this mode.
I hope you want to explore it.
Regards.

@antonym
Copy link
Member

antonym commented Aug 1, 2024

ProxyDHCP is already supported, if you drop a local-vars.ipxe in your local tftp root and set:

set use_proxydhcp_settings true

That will enable the proxydhcp code path:

https://github.com/netbootxyz/netboot.xyz/blob/development/roles/netbootxyz/templates/disks/netboot.xyz.j2#L31

@lars18th
Copy link
Author

lars18th commented Aug 1, 2024

Wow, that's great! I'll try it.

However, perhaps it has sense to add this description in the main README.md because this information has no visibility. And also I suggest to include a toggle in the UI to enable it, instead of a manual setting. Do you agree with that?

@antonym
Copy link
Member

antonym commented Aug 4, 2024

Moving to docs project to make a note to write some.

@antonym antonym transferred this issue from netbootxyz/docker-netbootxyz Aug 4, 2024
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

2 participants