Navigation Menu

Skip to content

Commit

Permalink
fix(bundles/publish): fix usage of CreateNewBundle without env variab…
Browse files Browse the repository at this point in the history
…les.

Added common.Get* functions for usage in CreateNewBundle call
  • Loading branch information
pallam authored and distorhead committed May 17, 2023
1 parent c66b846 commit 3e0e079
Showing 1 changed file with 5 additions and 4 deletions.
9 changes: 5 additions & 4 deletions cmd/werf/bundle/publish/publish.go
Expand Up @@ -15,6 +15,7 @@ import (
"helm.sh/helm/v3/pkg/cli/values"

"github.com/werf/logboek"

"github.com/werf/werf/cmd/werf/common"
"github.com/werf/werf/pkg/build"
"github.com/werf/werf/pkg/config"
Expand Down Expand Up @@ -408,10 +409,10 @@ func runPublish(ctx context.Context, imagesToProcess build.ImagesToProcess) erro
defer os.RemoveAll(bundleTmpDir)

bundle, err := wc.CreateNewBundle(ctx, bundleTmpDir, chartVersion, &values.Options{
ValueFiles: *commonCmdData.Values,
StringValues: *commonCmdData.SetString,
Values: *commonCmdData.Set,
FileValues: *commonCmdData.SetFile,
ValueFiles: common.GetValues(&commonCmdData),
StringValues: common.GetSetString(&commonCmdData),
Values: common.GetSet(&commonCmdData),
FileValues: common.GetSetFile(&commonCmdData),
})
if err != nil {
return fmt.Errorf("unable to create bundle: %w", err)
Expand Down

0 comments on commit 3e0e079

Please sign in to comment.