From ac827a7fffeb44dd66ad7fbc3a269fa00dc68510 Mon Sep 17 00:00:00 2001 From: Kir Kolyshkin Date: Tue, 6 Aug 2019 18:15:42 -0700 Subject: [PATCH] builder-next/patchImageConfig: nitpicks 1. Do not introduce another dt variable. 2. Wrap the error similar to all the others. Signed-off-by: Kir Kolyshkin --- builder/builder-next/exporter/writer.go | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/builder/builder-next/exporter/writer.go b/builder/builder-next/exporter/writer.go index b0f51c6ff5..c0aff65ece 100644 --- a/builder/builder-next/exporter/writer.go +++ b/builder/builder-next/exporter/writer.go @@ -78,9 +78,9 @@ func patchImageConfig(dt []byte, dps []digest.Digest, history []ocispec.History, } if cache != nil { - dt, err := json.Marshal(cache) + dt, err = json.Marshal(cache) if err != nil { - return nil, err + return nil, errors.Wrap(err, "failed to marshal cache") } m["moby.buildkit.cache.v0"] = dt }