From 46b2c2e31dffb626f0253edf66e2b1e491e3df1e Mon Sep 17 00:00:00 2001 From: Christian Kellner Date: Wed, 26 Jan 2022 17:12:14 +0000 Subject: [PATCH] worker: don't marshal manifest in byte[] from The manifest is of type distro.Manifest, which is an alias for a byte array, i.e. it is already in marshalled form. There is no need to marshal it again before passing it to osbuild. --- cmd/osbuild-worker/osbuild.go | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/cmd/osbuild-worker/osbuild.go b/cmd/osbuild-worker/osbuild.go index b8c1546a3..a769b031a 100644 --- a/cmd/osbuild-worker/osbuild.go +++ b/cmd/osbuild-worker/osbuild.go @@ -42,9 +42,9 @@ func RunOSBuild(manifest distro.Manifest, store, outputDirectory string, exports return nil, fmt.Errorf("error starting osbuild: %v", err) } - err = json.NewEncoder(stdin).Encode(manifest) + _, err = stdin.Write(manifest) if err != nil { - return nil, fmt.Errorf("error encoding osbuild pipeline: %v", err) + return nil, fmt.Errorf("error writing osbuild manifest: %v", err) } err = stdin.Close()