Re: server seed: remove acpid?

2018-11-03 Thread Robert Collins
On Thu., 1 Nov. 2018, 08:55 Andreas Hasenack  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.
>

That's good I think. Systemd has built in ACPI event support, better not be
processing sleep twice.

As far as I know removing it should be fine.

Rob
-- 
ubuntu-devel mailing list
ubuntu-devel@lists.ubuntu.com
Modify settings or unsubscribe at: 
https://lists.ubuntu.com/mailman/listinfo/ubuntu-devel


Re: server seed: remove acpid?

2018-11-01 Thread Steve Langasek
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/
slanga...@ubuntu.com vor...@debian.org


signature.asc
Description: PGP signature
-- 
ubuntu-devel mailing list
ubuntu-devel@lists.ubuntu.com
Modify settings or unsubscribe at: 
https://lists.ubuntu.com/mailman/listinfo/ubuntu-devel


server seed: remove acpid?

2018-10-31 Thread Andreas Hasenack
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?

-- 
ubuntu-devel mailing list
ubuntu-devel@lists.ubuntu.com
Modify settings or unsubscribe at: 
https://lists.ubuntu.com/mailman/listinfo/ubuntu-devel