Thursday 1 November 2018

Re: server seed: remove acpid?

On Wed, Oct 31, 2018 at 04:54:54PM -0300, Andreas Hasenack wrote:
> Hi,

> we are reviewing the server seeds
> (https://code.launchpad.net/~ubuntu-core-dev/ubuntu-seeds/+git/ubuntu/+ref/disco)
> and the acpid package came up. It's currently in the server seed,
> meaning it's installed by default.

> I did some tests in a VM, and a bare metal machine, running cosmic,
> and the power and suspend buttons, both from an usb keyboard and the
> box itself, worked just fine without acpid running.
> In fact, it's not even running on a default install unless the admin
> either enables it explicitly, or installs a package that populates
> /etc/acpi/events, or connects to the unix socket in /run/acpid.socket.

> Does anybody know what we would miss by not having it installed by default?

The above is what I expected to see, which is why I proposed it for dropping
from the seed. I expect systemd to handle the core acpi events (shutdown,
suspend) directly, with no need for acpid; so acpid is only needed if you
have other custom ACPI event handlers. If you have custom ACPI event
handlers on your server, you can install the acpid package as needed; but it
is no longer part of our platform and should not be installed by default.

(It is separately still being included by default in the desktop seed
because it is a dependency of acpi-support, which ships a handful of event
handlers that had never been ported elsewhere; it is worth revisiting
whether these are now handled elsewhere or if they should be dropped as
obsolete, but in any case that should not stop us from dropping acpid from
the server seed.)

--
Steve Langasek Give me a lever long enough and a Free OS
Debian Developer to set it on, and I can move the world.
Ubuntu Developer https://www.debian.org/
slangasek@ubuntu.com vorlon@debian.org