Support snap configuration

Bug #1596629 reported by Kyle Fazzari
20
This bug affects 2 people
Affects Status Importance Assigned to Milestone
snapd (Ubuntu)
Fix Released
Wishlist
Kyle Fazzari
Xenial
Fix Released
Wishlist
Unassigned
Yakkety
Fix Released
Wishlist
Unassigned

Bug Description

Two commands will be supported: `snap set key=value` and `snap get key`. `snap set` will cause the `configure` hook to be run within the snap in question, which gives it an opportunity to respond to those changes (including vetoing them). `snap get` will obtain the config values directly from snapd (no hooks involved).

Kyle Fazzari (kyrofa)
Changed in snapd (Ubuntu):
status: New → In Progress
importance: Undecided → Wishlist
assignee: nobody → Kyle Fazzari (kyrofa)
Revision history for this message
Kyle Fazzari (kyrofa) wrote :

Snap get/set commands (with no real functionality just yet): https://github.com/snapcore/snapd/pull/1828

Revision history for this message
Kyle Fazzari (kyrofa) wrote :

REST API behind the get/set commands: https://github.com/snapcore/snapd/pull/1866

Revision history for this message
Kyle Fazzari (kyrofa) wrote :
Revision history for this message
Kyle Fazzari (kyrofa) wrote :
Revision history for this message
Kyle Fazzari (kyrofa) wrote :
Revision history for this message
Kyle Fazzari (kyrofa) wrote :

Support for initial context and using it in REST API for snap set: https://github.com/snapcore/snapd/pull/2013

Revision history for this message
Kyle Fazzari (kyrofa) wrote :

Rename apply-config to config-changing: https://github.com/snapcore/snapd/pull/2016

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

Note that config-changing has been renamed to configure.

description: updated
Revision history for this message
Kyle Fazzari (kyrofa) wrote :
Changed in snapd (Ubuntu):
status: In Progress → Fix Committed
Revision history for this message
Andy Whitcroft (apw) wrote : Please test proposed package

Hello Kyle, or anyone else affected,

Accepted snapd into yakkety-proposed. The package will build now and be available at https://launchpad.net/ubuntu/+source/snapd/2.17.1+16.10 in a few hours, and then in the -proposed repository.

Please help us by testing this new package. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested, and change the tag from verification-needed to verification-done. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed. In either case, details of your testing will help us make a better decision.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance!

Changed in snapd (Ubuntu Yakkety):
status: New → Fix Committed
tags: added: verification-needed
Revision history for this message
Andy Whitcroft (apw) wrote :

Hello Kyle, or anyone else affected,

Accepted snapd into xenial-proposed. The package will build now and be available at https://launchpad.net/ubuntu/+source/snapd/2.17.1 in a few hours, and then in the -proposed repository.

Please help us by testing this new package. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested, and change the tag from verification-needed to verification-done. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed. In either case, details of your testing will help us make a better decision.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance!

Changed in snapd (Ubuntu Xenial):
status: New → Fix Committed
tags: added: verification-done
removed: verification-needed
Revision history for this message
Federico Gimenez (fgimenez) wrote :
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package snapd - 2.18+17.04ubuntu3

---------------
snapd (2.18+17.04ubuntu3) zesty; urgency=medium

  * disable ppc64el autopkgtest for now, it never worked but for
    some reason autopkgtest thinks it did at some point and asking
    for correction of the DB was not successful so far

 -- Michael Vogt <email address hidden> Fri, 02 Dec 2016 16:27:34 +0100

Changed in snapd (Ubuntu):
status: Fix Committed → Fix Released
Mathew Hodson (mhodson)
Changed in snapd (Ubuntu Xenial):
importance: Undecided → Wishlist
Changed in snapd (Ubuntu Yakkety):
importance: Undecided → Wishlist
Revision history for this message
Brian Murray (brian-murray) wrote :

Hello Kyle, or anyone else affected,

Accepted snapd into yakkety-proposed. The package will build now and be available at https://launchpad.net/ubuntu/+source/snapd/2.17.1+16.10ubuntu1 in a few hours, and then in the -proposed repository.

Please help us by testing this new package. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested, and change the tag from verification-needed to verification-done. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed. In either case, details of your testing will help us make a better decision.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance!

tags: removed: verification-done
tags: added: verification-needed
Revision history for this message
Launchpad Janitor (janitor) wrote :
Download full text (11.2 KiB)

This bug was fixed in the package snapd - 2.17.1ubuntu1

---------------
snapd (2.17.1ubuntu1) xenial; urgency=medium

  * drop xdelta dependency

snapd (2.17.1) xenial; urgency=medium

  * New upstream release, LP: #1637215:
    - release: os-release on core has changed
    - tests: /dev/ptmx does not work on powerpc, skip here
    - docs: moved to github.com/snapcore/snapd/wiki (#2258)
    - debian: golang is not installable on powerpc, use golang-any

snapd (2.17) xenial; urgency=medium

  * New upstream release, LP: #1637215:
    - overlord/ifacestate: add unit tests for undo of setup-snap-
      security (#2243)
    - daemon,overlord,snap,tests: download to .partial in final dir
      (#2237)
    - overlord/state: marshaling tests for lanes (#2245)
    - overlord/state: introduce state lanes (#2241)
    - overlord/snapstate: fix revert+refresh (#2224)
    - interfaces/sytemd: enable/disable generated service units (#2229)
    - many: fix incorrect security files generation on undo
    - overlord/snapstate: add dynamic snapdX.Y assumes (#2227)
    - interfaces: network-manager: give slot full read-write access to
      /run/NetworkManager
    - docs: update the name of the command for the cross-build
    - overlord/snapstate: fix missing argument to Noticef
    - snapstate: ensure gadget/core/kernel can not be disabled (#2218)
    - asserts: limit to 1y only if len(models) == 0 (#2219)
    - debian: only install share/locale if available (missing on
      powerpc)
    - overlrod/snapstate: fix revert followed by refresh to old-current
      (#2214)
    - interfaces/builtin: network-manager and bluez can change hostname
      (#2204)
    - snap: switch the auto-import dir to /run/snapd/auto-import
    - docs: less details about cloud.cfg as requested in trello (#2206)
    - spread.yaml: Ensure ubuntu user has passwordless sudo for
      autopkgtests (#2201)
    - interfaces/builtin: add dcdbas-control interface
    - boot: do not set boot to try mode if the revision is unchanged
    - interfaces: add shutdown interface (#2162)
    - interfaces: add system-power-control interface
    - many: use the new systemd backend for configuring GPIOs
    - overlord/ifacestate: setup security for slots before plugs
    - snap: spool assertion candidates if snapd is not up yet
    - store,daemon,overlord: download things to a partials dir
    - asserts,daemon: implement system-user-authority header/concept
    - interfaces/builtin: home base declaration rule using on-classic
      for its policy
    - interfaces/builtin: finish decl based checks
    - asserts: bump snap-declaration to allow signing with new-style
      plugs and slots
    - overlord: checks for kernel installation/refresh based on model
      assertion and previous kernel
    - tests/lib/fakestore: fix logic to distinguish assertion not found
      errors
    - client: add a few explicit error types (around the request cycle)
    - tests/lib/fakestore/cmd/fakestore: make it log, and fix a typo
    - overlord/snapstate: two bugs for one
    - snappy: disable auto-import of assertions on classic (#2122)
    - overlord/snapstate: move trash cleanup to a cleanup handler
      (#2173)
    -...

Changed in snapd (Ubuntu Xenial):
status: Fix Committed → Fix Released
Revision history for this message
Federico Gimenez (fgimenez) wrote :

Verified on yakkety:

$ apt-cache policy snapd
snapd:
  Installed: 2.17.1+16.10ubuntu1
  Candidate: 2.17.1+16.10ubuntu1
  Version table:
 *** 2.17.1+16.10ubuntu1 500
        500 http://archive.ubuntu.com/ubuntu yakkety-proposed/main amd64 Packages
        100 /var/lib/dpkg/status
     2.16+16.10ubuntu1.2 500
        500 http://es.archive.ubuntu.com/ubuntu yakkety-updates/main amd64 Packages
     2.16+16.10ubuntu1 500
        500 http://es.archive.ubuntu.com/ubuntu yakkety/main amd64 Packages
$ sudo snap install --candidate spread-cron
spread-cron (candidate) 1.0 from 'canonical' installed
$ sudo snap set spread-cron username=blabla
$

tags: added: verification-done
removed: verification-needed
Revision history for this message
Federico Gimenez (fgimenez) wrote :

The final "snap get" call was missing in the previous paste:

$ sudo snap get spread-cron username
blabla

Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package snapd - 2.20.1+16.10ubuntu2

---------------
snapd (2.20.1+16.10ubuntu2) yakkety; urgency=medium

  * New upstream release, LP: #1648520
    - spread.yaml: update for ppc64el, i386, armhf

 -- Michael Vogt <email address hidden> Tue, 03 Jan 2017 22:16:08 +0100

Changed in snapd (Ubuntu Yakkety):
status: Fix Committed → Fix Released
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.