snap: add support user-sessions from snaps #2752

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
6 participants
Collaborator

mvo5 commented Jan 31, 2017

The ubuntu-personal team asked about support for user sessions. It seems we have most pieces in place. Here is a minimal branch, mostly to start a discussion (for a final branch we will need a bunch more tests).

The strawman in this branch is to use:

name: snap
apps:
 daemon1:
  command: foo --daemon
  daemon: simple
  user-service: true

alternatively we could use something like: daemon-scope: {user,system} or just scope: {user,system}. Or something entirely differently, I'm open for ideas :)

Contributor

zyga commented Jan 31, 2017

Looks sensible on a quick look.

Contributor

vosst commented Jan 31, 2017

@mvo5 Thanks for the PR. I tend to like scope: {system, user} better than a bool flag. Mainly because user-service: false does not necessarily imply system-service.

Contributor

morphis commented Feb 7, 2017

@mvo5 Is this supposed to work on classic as well or just for the purpose of an all-snap system? Wondering otherwise how this would fit into the user session managed by upstart on 16.04.

Contributor

vosst commented Feb 7, 2017

@morphis that's an interesting point. We would need a systemd instance managed by upstart, much like in the trusty scenario. I'll take the action to investigate into Trusty's systemd and check on its support for user sessions.

Member

chipaca commented Feb 10, 2017

a bare scope might make people coming from mobile think of something else entirely. Otherwise agreed it's better than the boolean.

@zyga zyga added the Blocked label Feb 13, 2017

Contributor

zyga commented Feb 13, 2017

I've added the blocked tag based on discussion above about 14.04 systemd running as a helper for user sessions.

Contributor

niemeyer commented Mar 10, 2017

This is a pretty trivial change and has been blocked for quite a while on unrelated details. I'm closing it for now, but please reopen when we actually intend to perform the change.

@niemeyer niemeyer closed this Mar 10, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment