Skip to content

kea: add a kea-hook-pingcheck package#29066

Open
aidanharris wants to merge 2 commits intoopenwrt:masterfrom
aidanharris:kea-pingcheck
Open

kea: add a kea-hook-pingcheck package#29066
aidanharris wants to merge 2 commits intoopenwrt:masterfrom
aidanharris:kea-pingcheck

Conversation

@aidanharris
Copy link
Copy Markdown

@aidanharris aidanharris commented Apr 4, 2026

📦 Package Details

Maintainer: @pprindeville, @nmeyerhans

Description:
This is built on top of #29065 but I can rebase it if necessary. It's a simple change.


🧪 Run Testing Details

  • OpenWrt Version: HEAD
  • OpenWrt Target/Subtarget: x86_64/generic
  • OpenWrt Device:

✅ Formalities

  • I have reviewed the CONTRIBUTING.md file for detailed contributing guidelines.

The Run Script hook library adds support for calling an external script
for specific packet-processing hook points.

Signed-off-by: Aidan Harris <me@aidanharr.is>
The Ping check hook library adds the abilitity to perform a "ping check"
of a candidate IPv4 address prior to offering it to a DHCP client.

Signed-off-by: Aidan Harris <me@aidanharr.is>
@pprindeville
Copy link
Copy Markdown
Member

Is this being upstreamed? Why don't we just wait for it to hit us?

@aidanharris
Copy link
Copy Markdown
Author

@pprindeville These hooks are already upstream and built they're just not installed anywhere on OpenWRT.

I think the package needs work to install all of them. These two hooks are what most interest me because they provide useful functionality. But I can try to package the rest too if there's interest. I just didn't want to do a bunch of work that would go to waste.

@pprindeville
Copy link
Copy Markdown
Member

@pprindeville These hooks are already upstream and built they're just not installed anywhere on OpenWRT.

I think the package needs work to install all of them. These two hooks are what most interest me because they provide useful functionality. But I can try to package the rest too if there's interest. I just didn't want to do a bunch of work that would go to waste.

Package it and I'll merge it.

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