networkd-dispatcher should read from /usr/lib, not /etc
Bug #1765152 reported by
Steve Langasek
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| networkd-dispatcher (Ubuntu) |
Fix Released
|
Critical
|
Unassigned | ||
| Bionic |
Fix Released
|
Critical
|
Unassigned | ||
Bug Description
networkd-dispatcher upstream only supports reading hooks from a single directory and defaults to /etc. We don't want packages to have to ship more conffiles (with potentially buggy behavior when the package is removed but not purged) in /etc.
We should change the package to read from /usr/lib instead of from /etc by default before this goes out the door for 18.04. We can then later improve the package to support reading hooks from multiple locations à la systemd itself, to reintroduce support for local hooks in /etc. But we should not ship this in a way that requires packages to integrate by dropping hooks in /etc.
Related branches
~paelzer/ubuntu/+source/chrony:bionic-lp1764357
- Steve Langasek (community): Needs Fixing
- Nish Aravamudan (community): Approve
- Canonical Server packageset reviewers: Pending requested
- git-ubuntu developers: Pending requested
-
Diff: 199 lines (+136/-1)7 files modifieddebian/changelog (+13/-0)
debian/control (+1/-0)
debian/links (+5/-0)
debian/patches/lp-1718227-ignore-non-up-down-events-in-nm-dispatcher.patch (+32/-0)
debian/patches/lp-1718227-nm-dispatcher-for-networkd.patch (+67/-0)
debian/patches/series (+2/-0)
debian/postrm (+16/-1)
| Changed in networkd-dispatcher (Ubuntu): | |
| importance: | Undecided → Critical |
| status: | New → Triaged |
Revision history for this message
| Christian Ehrhardt (paelzer) wrote | #1 |
Revision history for this message
| Christian Ehrhardt (paelzer) wrote | #2 |
Revision history for this message
| Julian Andres Klode (juliank) wrote | #3 |
| Changed in networkd-dispatcher (Ubuntu Bionic): | |
| status: | Triaged → Fix Committed |
Revision history for this message
| Christian Ehrhardt (paelzer) wrote | #4 |
Revision history for this message
| Julian Andres Klode (juliank) wrote | #5 |
| Changed in networkd-dispatcher (Ubuntu Bionic): | |
| status: | Fix Committed → Fix Released |
| tags: | added: id-5ad78a069c1c90d680d71475 |
To post a comment you must log in.
Please post here what the new path exactly will be like so that I can adapt my changes depending on networkd-dispatcher without having to wait until it fully lands to check in the source then.
So as soon as you are sure on the new paths, let me know here - thanks in advance.