Mir-kiosk:mir-libs Auto-Connection

AlbertA 2017-10-12 19:02:42 UTC #1

mir-kiosk snap: https://dashboard.snapcraft.io/dev/snaps/6225/

We want to deprecate the use of mir-libs content snap as the provider of mir-libs content and consolidate it instead with mir-kiosk snap itself. This will be bit less confusing to users of mir-kiosk.

mir-libs content snap currently has Auto-connection enabled for its mir-libs slot. We would like mir-kiosk:mir-libs to have Auto-Connection enabled as mir-libs content snap has.

Thanks!

Mir-kiosk. does it work?

jdstrand 2017-10-12 19:08:21 UTC #2

I’ve not looked into this yet, but wanted to quickly say that the auto-connection for the old mir-libs snap happened before the critical thinking regarding auto-connection across publishers and discussed elsewhere (if unfamiliar with that thread, it is best summarized in this video). As such, we should be evaluating this as a new request.

AlbertA 2017-10-24 14:38:45 UTC #3

Is there any update on this?

jdstrand 2017-10-26 19:30:56 UTC #4

Sorry for the delay. Looking at your snap.yaml, you have this:

slots:
  mir-libs:
    content: mir0
    interface: content
    read:
    - usr/lib

It seems you may not have considered my previous comment and the new thinking around the content interface. In the snap’s current form, -1.

Assuming you built mir-kiosk on 16.04, if you update the snap to use this (modeled after the gnome-3-26-1604 snap):

slots:
  mir-libs-1604:
    content: mir0-1604
    interface: content
    read:
    - usr/lib

(or something similarly clear that it was built using 16.04 environment), then it would ok to grant auto-connection. Please see the links in my previous response if you have questions as to why this is required.

AlbertA 2017-11-02 16:09:25 UTC #5

Sounds reasonable,

We’ll do that thanks!

jdstrand 2017-11-08 13:38:07 UTC #6

Note to voters, as of today, the snap’s slot has not yet changed so please refrain from voting.

jdstrand 2017-11-15 15:30:09 UTC #7

@AlbertA - what timeframe are you expecting this to be completed?

AlbertA 2017-11-15 16:08:47 UTC #8

Hey, just noticed launchpad failed to upload to the store. It should be updated now.

jdstrand 2017-11-15 18:56:09 UTC #9

Thanks!

Here is the snap yaml for the slot:

slots:
  mir-libs-1604:
    content: mir0-1604
    interface: content
    read:
    - usr/lib

+1 from me.

@niemeyer, can you review the above and vote/comment?

niemeyer 2017-11-21 23:39:09 UTC #10

Sounds good to me as well.

As a minor note, we’re improving the content interface so this may benefit from some changes in the near future. That said, we of course won’t break the way it currently behaves, so if you need the interface right away, that’s the way to go.

Again, +1, and thanks for addressing the versioning detail, @jdstrand.

jdstrand 2017-11-27 18:50:32 UTC #11

2 votes for, 0 against. Granting the auto-connection. This is now live.