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

ffh.dhcp_server: switch to Kea DHCP server #110

Closed
wants to merge 1 commit into from
Closed

Conversation

CodeFetch
Copy link
Contributor

Kea is the successor of ISC DHCP and will hopefully not cause
problems as like issue #109.

roles/ffh.dhcp_server/tasks/main.yml Show resolved Hide resolved
"loggers": [{
"name": "kea-dhcp4",
"output_options": [{ "output": "syslog:dhcpd", }],
"severity": "WARN"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What does WARN include? Do we store more information than we have to with this feature?

roles/ffh.dhcp_server/tasks/per_domain.yml Show resolved Hide resolved
@CodeFetch CodeFetch force-pushed the pr_kea branch 3 times, most recently from 79f3a2b to c8c322e Compare January 26, 2021 00:03
Kea is the successor of ISC DHCP and will hopefully not cause
problems as like issue #109.
@CodeFetch
Copy link
Contributor Author

Closed as Kea is badly maintained in Debian:
https://tracker.debian.org/pkg/isc-kea

@CodeFetch CodeFetch closed this Jan 26, 2021
@1977er
Copy link
Member

1977er commented Jan 26, 2021

Why do we care? We are using other software that is not even available from Debian. Or maintained at all.

@AiyionPrime
Copy link
Member

I don't think this should stop us.
In case we did care, we might even better help the maintainer out.

@AiyionPrime
Copy link
Member

I've looked into it;
this does not even look like the maintainers were sleeping at all, but as if debians release cycle is just awfully slow.
I'd like to reopen this. Version 1.1 would be fine for me for now, its security record is not that bad.
Version 1.4.0 has had a leak but that's taken care of already.

https://qa.debian.org/excuses.php?package=isc-kea

@AiyionPrime AiyionPrime reopened this Jan 26, 2021
@AiyionPrime
Copy link
Member

In case we wanted to kick packages this one would be a good start.
https://tracker.debian.org/pkg/log4cplus

@CodeFetch
Copy link
Contributor Author

CodeFetch commented Jan 26, 2021

I'm against choosing Kea and will not invest more time into it. That's why I closed it. If you want to support Kea, feel free to take over the PR. I've reformulated @Manawyrm pros/cons to reflect my experience.

Kea DHCP: 
- badly-written documentation (steep learning curve/hard to find information)
+ JSON-style
- JSON-format is not well thought-through, because aim was to mimic ISC config format
+ modern
- not included in a recent version in Debian stable
# pretends to handle large setups fine
# pretends to be a professional enterprise-solution
- not so interesting for open-source developers (therefore potentially badly reviewed)
- the debian-shipped version is so old it doesn't even allow comments in config files

dnsmasq:
+ has the most active developer community
+ we know some of the contributors as it is the standard on OpenWrt and will therefore get better support when we have issues
# Manawyrm says does not handle large installation well, but I can't find any issues by other people
- bad security track record
+ someone cared enough to fix security

ddhcpd: 
+ runs directly on the gluon nodes
+ quick response times
- needs secondary fallback DHCP for old nodes/without autoupdater
- does likely not include optimizations for increasing compatibility with e.g. Windows
- needs testing

@CodeFetch CodeFetch closed this Jan 26, 2021
@CodeFetch
Copy link
Contributor Author

It just feels wrong. Hope you understand, that I don't have any motivation to try Kea anymore. Maybe sometime in the future, when there is a stable release in Debian.

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.

3 participants