docker-container: restart-policy opt by crazy-max · Pull Request #1271 · docker/buildx (original) (raw)

LGTM.

Personally, not sure about defaulting to always, I quite like the default lazy behavior with restart-policy=no, since I can sometimes have a lot of builders, which might not all actually be running. But, also, this would then match the kubernetes driver behavior, where the replicas are always running. If we do switch the default, we should default to unless-stopped I think, since buildx stop will stop the container, which then shouldn't automatically restart.