Skip to content

Commit e9f76e2

Browse files
committed
test(snap): pre-pull default image more reliably
We need to give docker a moment to finish setting up. We an also add a docker system group while we are at it. With this setting it is worth to keep prepared images around in CI but the time advantage is worth it. Signed-off-by: Zygmunt Krynicki <zygmunt.krynicki@canonical.com>
1 parent 6e77fc0 commit e9f76e2

1 file changed

Lines changed: 11 additions & 8 deletions

File tree

.image-garden.mk

Lines changed: 11 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -3,19 +3,24 @@
33

44
default_sandbox_image=ghcr.io/nvidia/openshell-community/sandboxes/base:latest
55

6+
define install_and_setup_docker
7+
- addgroup --system docker
8+
- snap install docker
9+
- while [ ! -S /run/docker.sock ]; do sleep 3; done
10+
- "snap run docker pull $(default_sandbox_image)"
11+
endef
12+
613
define UBUNTU_CLOUD_INIT_USER_DATA_TEMPLATE
714
$(CLOUD_INIT_USER_DATA_TEMPLATE)
815
- snap wait system seed.loaded
9-
- snap install docker
10-
- snap run docker pull $(default_sandbox_image)
16+
$(install_and_setup_docker)
1117
endef
1218

1319
define DEBIAN_CLOUD_INIT_USER_DATA_TEMPLATE
1420
$(CLOUD_INIT_USER_DATA_TEMPLATE)
1521
- systemctl enable --now snapd.socket snapd.service snapd.apparmor.service
1622
- snap wait system seed.loaded
17-
- snap install docker
18-
- snap run docker pull $(default_sandbox_image)
23+
$(install_and_setup_docker)
1924
packages:
2025
- snapd
2126
endef
@@ -26,8 +31,7 @@ $(CLOUD_INIT_USER_DATA_TEMPLATE)
2631
- systemctl enable --now snapd.socket
2732
- snap wait system seed.loaded
2833
- sudo ln -s /var/lib/snapd/snap /snap
29-
- snap install docker
30-
- snap run docker pull $(default_sandbox_image)
34+
$(install_and_setup_docker)
3135
endef
3236

3337
define CENTOS_CLOUD_INIT_USER_DATA_TEMPLATE
@@ -37,6 +41,5 @@ $(CLOUD_INIT_USER_DATA_TEMPLATE)
3741
- systemctl enable --now snapd.socket snapd.service
3842
- snap wait system seed.loaded
3943
- sudo ln -s /var/lib/snapd/snap /snap
40-
- snap install docker
41-
- snap run docker pull $(default_sandbox_image)
44+
$(install_and_setup_docker)
4245
endef

0 commit comments

Comments
 (0)