snapd layout feature not supported

Bug #1754639 reported by Ondrej Kubik
28
This bug affects 4 people
Affects Status Importance Assigned to Milestone
Snapcraft
Fix Released
Wishlist
Kyle Fazzari

Bug Description

snapd landed new feature "layout" which needs to be supported by snapcraft

layout allows file system overlay, it is defined in snapcraft.yaml
Such a definition triggers error from snapcraft build
Issues while validating snapcraft.yaml: Additional properties are not allowed ('layout' was unexpected)

Ondrej Kubik (ondrak)
description: updated
Revision history for this message
Kyle Fazzari (kyrofa) wrote :

Thanks ondra! Can you please include the snapd version for when this was introduced? Or is it still master?

Revision history for this message
Ondrej Kubik (ondrak) wrote :

layout has been now for some time in edge channel.
I verified functionality with 2.31.1+git607.a15e601~ubuntu16.04.1

Revision history for this message
Gerry Boland (gerboland) wrote :

I've also verified it with snapd in the edge channel (same rev as Ondrej). It is an extremely useful feature, allows me remove LD_PRELOAD shims which redirect file access, which was causing problems with some software.

Revision history for this message
Gustavo Niemeyer (niemeyer) wrote :

Please note that on the first release this feature will be blocked behind a feature flag that needs to be enabled in snapd itself by doing:

$ snap set core experimental.layouts=true

Snaps that use that feature will also be blocked for manual review in the store.

Both of those measures aim at giving us some more time of stabilization before we promise the feature as ready for use and with the usual syntax stability guarantees. That was deemed appropriate for this case due to the underlying complexity involved in getting it to work.

Revision history for this message
Ondrej Kubik (ondrak) wrote :

Gustavo that is fine, and indeed we should be careful when landing layouts.
I believe snapcraft allowing actual snap building is something which should happen in parallel, rather then start implementation once layouts land in stable

Revision history for this message
Sergio Schvezov (sergiusens) wrote :

This should probably go with the passthrough feature.

Changed in snapcraft:
status: New → Triaged
importance: Undecided → High
assignee: nobody → Christian Dywan (kalikiana)
importance: High → Wishlist
Kyle Fazzari (kyrofa)
tags: added: 18.10-snapd-integration
Changed in snapcraft:
milestone: none → 3.0
assignee: Christian Dywan (kalikiana) → Kyle Fazzari (kyrofa)
Revision history for this message
Kyle Fazzari (kyrofa) wrote :
Changed in snapcraft:
status: Triaged → In Progress
Revision history for this message
Kyle Fazzari (kyrofa) wrote :
Changed in snapcraft:
status: In Progress → Fix Committed
Changed in snapcraft:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

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