fix: should ignore nil client by morlay · Pull Request #686 · docker/buildx (original) (raw)

Multi-node build support and the underlying clients iteration was first introduced while resolving drivers (#37). Client nil check was already there

buildx/build/build.go

Lines 283 to 285 in7d312ea

but was left out to be included in #535.

but perhaps we should also look into why this slice contains nil values in the first place,

Yes agree, might be an issue with the context dialer while creating the buildkit client for docker-container driver. Will take a look.