'daemon: dbus' is incomplete

Bug #1649399 reported by Jamie Strandboge
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
snapd
In Progress
Undecided
James Henstridge

Bug Description

'daemon: dbus' was part of Ubuntu Core 15.04, it was only for system services and it was removed in series 16 in favor of 'interfaces'. As of now,

1. snapd allows specifying 'daemon: dbus' but when you do it generates an incomplete systemd service file:

/usr/lib/snapd/snapd[2329]: task.go:303: DEBUG: 2016-12-12T14:33:18-06:00 ERROR [start snap.test-hello-dbus.test-hello-dbusd-system.service] failed with exit status 1: Failed to start snap.test-hello-dbus.test-hello-dbusd-system.service: Unit snap.test-hello-dbus.test-hello-dbusd-system.service is not loaded properly: Invalid argument.#012See system logs and 'systemctl status snap.test-hello-dbus.test-hello-dbusd-system.service' for details.

2. snapcraft doesn't allow you to specify it (which is good since the implementation doesn't work correctly)

3. http://snapcraft.io/docs/snaps/metadata incorrectly lists it

4. https://github.com/snapcore/snapd/wiki/Snap-format also incorrectly lists it

I adjusted '4' just now. I'm not sure where to file a bug for '3'.

For '1', we could consider updating the 'dbus' interface in some manner to have snapd properly tie in with systemd. It already supports 'system' dbus names, generates a dbus bus policy file and adjusts security policy such that a daemon started without Type=dbus will work. It should therefore be possible to let the developer specify 'daemon: dbus' under certain conditions. This needs some design since the systemd service files allow only one BusName, but snapd supports a daemon to specify multiple slots for acquiring multiple dbus well-known names.

Tags: snap-docs
David Callé (davidc3)
tags: added: snap-docs
Revision history for this message
David Callé (davidc3) wrote :

Regarding 3.

Adding the snap-docs tag to track the snapcraft.io bug.
Fixed in docs (next deployment tomorrow) and setting up this page to import from the wiki.

Revision history for this message
Jamie Strandboge (jdstrand) wrote :

FYI, snap/validate.go still allows 'daemon: dbus'. The dbus interface still only does 'Type: simple'. It is removed from all documentation and the review-tools don't allow it.

Zygmunt Krynicki (zyga)
affects: snappy → snapd
Revision history for this message
Ian Johnson (anonymouse67) wrote :

Work on supporting daemon: dbus is ongoing in PR https://github.com/snapcore/snapd/pull/6258

Changed in snapd:
status: New → In Progress
Zygmunt Krynicki (zyga)
Changed in snapd:
assignee: nobody → James Henstridge (jamesh)
Revision history for this message
Zygmunt Krynicki (zyga) wrote :

This feature is still in progress but is blocked behind some other branches.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.