networkd-dispatcher should read from /usr/lib, not /etc

Bug #1765152 reported by Steve Langasek on 2018-04-18
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
networkd-dispatcher (Ubuntu)
Critical
Unassigned
Bionic
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

Steve Langasek (vorlon) on 2018-04-18
Changed in networkd-dispatcher (Ubuntu):
importance: Undecided → Critical
status: New → Triaged

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.

Also I think I can assume this to be 1.7-0ubuntu3 (as the request was "before 18.04 goes out").
Let me version depend on this ....

Julian Andres Klode (juliank) wrote :

/usr/lib/networkd-dispatcher/*.d/ are in 1.7-0ubuntu3. I just uploaded that, but I forgot to add a LP: tag to the changelog in all the excitement.

Changed in networkd-dispatcher (Ubuntu Bionic):
status: Triaged → Fix Committed

Tested your new version from proposed with my hooks from the ppa.
Working well together, really just a path change.

Thanks

Julian Andres Klode (juliank) wrote :

1.7-0ubuntu3 released.

Changed in networkd-dispatcher (Ubuntu Bionic):
status: Fix Committed → Fix Released
tags: added: id-5ad78a069c1c90d680d71475
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers