Skip to content

kea: Add support for plugin preprocessors (importers) #27439

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.

Sign up for GitHub

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

Jump to bottom
Draft
wants to merge 2 commits into
base: master
Choose a base branch
from
Draft

kea: Add support for plugin preprocessors (importers) #27439

wants to merge 2 commits into from

Conversation

@pprindeville
Copy link
Member

@pprindeville pprindeville commented Sep 13, 2025 •
edited
Loading

📦 Package Details

Maintainer: @pprindeville, @nmeyerhans
Description:
Add importer for legacy ISC-DHCP UCI.

🧪 Run Testing Details

  • OpenWrt Version:
    HEAD
  • OpenWrt Target/Subtarget:
    x86_64/generic
  • OpenWrt Device:
    Supermicro SYS-5018D-FN4T

✅ Formalities

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

If your PR contains a patch:

  • It can be applied using git am
  • It has been refreshed to avoid offsets, fuzzes, etc., using
  • It is structured in a way that it is potentially upstreamable

@pprindeville pprindeville marked this pull request as draft September 13, 2025 17:39
@pprindeville
Copy link
Member Author

pprindeville commented Sep 14, 2025

This seems unrelated:

2025-09-13T18:48:27.7163801Z checking for Boost system library... no
2025-09-13T18:48:27.7165214Z configure: error: Linking with  -lboost_system is not enough: please make sure libboost_system is installed in an expected location; Check config.log for details, you may be missing other libraries.
2025-09-13T18:48:27.7713680Z make[2]: *** [Makefile:291: /builder/build_dir/target-x86_64_musl/kea-2.6.4/.configured_68b329da9893e34099c7d8ad5cb9c940] Error 1
2025-09-13T18:48:27.7714786Z make[2]: Leaving directory '/feed/net/kea'
2025-09-13T18:48:27.7717118Z time: package/feeds/packages_ci/kea/compile#34.75#6.64#41.45
2025-09-13T18:48:27.7725733Z     ERROR: package/feeds/packages_ci/kea failed to build.
2025-09-13T18:48:27.7730921Z make[1]: *** [package/Makefile:183: package/feeds/packages_ci/kea/compile] Error 1

@pprindeville pprindeville force-pushed the add-kea-plugin branch 2 times, most recently from ad18727 to bd375ba Compare September 24, 2025 23:41
@pprindeville pprindeville changed the title Add kea plugin Add support for plugin preprocessors (importers) Sep 25, 2025
@pprindeville pprindeville force-pushed the add-kea-plugin branch 2 times, most recently from 3c01085 to dd17e50 Compare October 29, 2025 20:20
@Neustradamus
Copy link

Neustradamus commented Nov 10, 2025

Important: Kea replaces ISC DHCP.

@pprindeville pprindeville force-pushed the add-kea-plugin branch 5 times, most recently from 57a1254 to 48eb289 Compare November 15, 2025 03:08
feckert
feckert reviewed Nov 24, 2025
View reviewed changes
@pprindeville pprindeville changed the title Add support for plugin preprocessors (importers) kea: Add support for plugin preprocessors (importers) Nov 24, 2025
@pprindeville pprindeville force-pushed the add-kea-plugin branch 2 times, most recently from e70013b to 6f4df2c Compare November 24, 2025 22:54
@pprindeville
kea: add support for importers that synthesize config files
c4f9c17 
Configuring Kea JSON files is not trivial, and this might impede
the adoption of Kea as a DHCP server.  There are, however, many
users who used its predecessor ISC-DHCP, at least for DHCPv4.
A filter could ingest the legacy UCI and synthesize a JSON config
file for Kea DHCPv4.

Signed-off-by: Philip Prindeville <[email protected]>
@pprindeville
kea: Add importer for legacy isc-dhcp-v4 UCI configs
978a871 
Now that ISC-DHCP is EOLs, users might want to transparently
to the functionality of Kea.  This supports most of the
functionality of ISC-DHCP for v4.

Signed-off-by: Philip Prindeville <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@feckert feckert feckert left review comments

@nmeyerhans nmeyerhans Awaiting requested review from nmeyerhans

@jow- jow- Awaiting requested review from jow-

@nbd168 nbd168 Awaiting requested review from nbd168

@dangowrt dangowrt Awaiting requested review from dangowrt

@aparcar aparcar Awaiting requested review from aparcar

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@pprindeville @Neustradamus @feckert