Skip to content

Commit fca1828

Browse files
authored
fix: Restart buildkit after containerd when provisioning (#461)
Issue #, if available: Fixes #412 *Description of changes:* - Previously, only containerd was restarted after configuring it to use the data on the persistent disk. This changes the UUID of the server worker. BuildKit also needs to be restarted to use the proper UUID. See issue for why this is important. *Testing done:* - Local testing - [x] I've reviewed the guidance in CONTRIBUTING.md #### License Acceptance By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license. --------- Signed-off-by: Justin Alvarez <[email protected]>
1 parent 1b3ad94 commit fca1828

File tree

1 file changed

+3
-1
lines changed

1 file changed

+3
-1
lines changed

Diff for: finch.yaml

+3-1
Original file line numberDiff line numberDiff line change
@@ -176,7 +176,9 @@ provision:
176176
mkdir -p ~/.local/share/cni
177177
sudo mount --bind /mnt/lima-finch/cni ~/.local/share/cni
178178
179-
sudo systemctl restart containerd.service
179+
# Make sure buildkit is restarted with containerd, so it uses the correct UUID
180+
sudo systemctl add-requires buildkit.service containerd.service
181+
sudo systemctl restart containerd.service
180182
181183
# Probe scripts to check readiness.
182184
# 🟢 Builtin default: null

0 commit comments

Comments
 (0)