curtin-dev team mailing list archive

[Ryan Harper <rharper@xxxxxxxxxxx>]

Ryan Harper has proposed merging ~raharper/curtin:fix/curtainer-dont-wait-for-snap into curtin:master.

Commit message:
tools/curtainer: do not wait for snapd.seeded.service

The curtainer tool is used in CI and other places to extract a certain
curtin source from the archive and then is used to pack/unpack the
curtin code in the correct environment.  We have no need or use of
snapd in this container which currently blocks for longer than
curtainer waits for the system to boot due to waiting for snaps to
download and install.


Requested reviews:
  curtin developers (curtin-dev)

For more details, see:
https://code.launchpad.net/~raharper/curtin/+git/curtin/+merge/389306
-- 
Your team curtin developers is requested to review the proposed merge of ~raharper/curtin:fix/curtainer-dont-wait-for-snap into curtin:master.

diff --git a/tools/curtainer b/tools/curtainer
index a6fdc1b..91905a5 100755
--- a/tools/curtainer
+++ b/tools/curtainer
@@ -69,6 +69,8 @@ while [ ! -e /run/cloud-init/result.json ] && i=$(($i+1)); do
     [ $i -ge $max ] && exit 1
     [ "$debug" = "0" ] || echo -n .
     sleep 1
+    # do not wait for snapd to seed this image
+    systemctl stop snapd.seeded.service |:
 done
 [ "$debug" = "0" ] || echo "[done after $i]"
 exit 0