Skip to content

Commit

Permalink
fix: Restart buildkit after containerd when provisioning (#461)
Browse files Browse the repository at this point in the history
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]>
  • Loading branch information
pendo324 authored Aug 10, 2023
1 parent 1b3ad94 commit fca1828
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion finch.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -176,7 +176,9 @@ provision:
mkdir -p ~/.local/share/cni
sudo mount --bind /mnt/lima-finch/cni ~/.local/share/cni
sudo systemctl restart containerd.service
# Make sure buildkit is restarted with containerd, so it uses the correct UUID
sudo systemctl add-requires buildkit.service containerd.service
sudo systemctl restart containerd.service
# Probe scripts to check readiness.
# 🟢 Builtin default: null
Expand Down

0 comments on commit fca1828

Please sign in to comment.