Skip to content

Update config.h #169

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

JamesOBenson
Copy link

Add support for newer Pi with MAC-addresses that start with 28:cd:c1, 2c:cf:67, 3a:35:41

Fixes issue #166

Add support for newer Pi with MAC-addresses that start with 28:cd:c1, 2c:cf:67, 3a:35:41
@HackXIt
Copy link

HackXIt commented Feb 12, 2025

Unfortunately this fix is not working for me, I still don't see the newer clients, is there something else I've missed?

I checked out the PR, rebuilt the package, installed the package, rebooted the server and reconfigured the DHCP.

But my Raspberry Pi's with OUI 2c:cf:67 still get the response:

DHCPDISCOVER(eth1) 2c:cf:67:b3:4a:14 no address available

Or to be more clear, PiServer does not create the folder for TFTP for that device get a bootloader.

Any help is greatly appreciated!

It's sad that Raspberry Pi does not maintain this project properly, it is actually a very nice and tailored solution.

@HackXIt
Copy link

HackXIt commented Feb 12, 2025

Ok, I managed to fix it. piserver.cpp was missing the following for dnsmasq to hand out an IP Address:

fs << "pxe-service=tag:piserver,0,\"Raspberry Pi Boot\"" << endl;

// ADD THIS
fs << "dhcp-vendorclass=set:piserver,PXEClient" << endl;

for (auto &kv : _hosts)
{
    if (kv.second->distro())
    {
        fs << "dhcp-host="+kv.second->mac()+",set:piserver" << endl;
    }
}

With that in place, the client successfully showed up in the GUI and I could successfully boot my RPi4.. Hoooraaay.

Please for the love of god, somebody update this repo.

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