Combine builder options

Signed-off-by: Boris Kreitchman <bkreitch@gmail.com>
pull/5197/head
Boris Kreitchman 2 months ago
parent 3c08926150
commit 3226043088

@ -111,20 +111,10 @@ func buildKsCmdRun(cmd *cobra.Command, args []string) (err error) {
} }
} }
var options []build.BuilderOptionFunc
var builder *build.Builder var builder *build.Builder
if buildKsArgs.dryRun {
builder, err = build.NewBuilder(name, buildKsArgs.path, options = append(options,
build.WithTimeout(rootArgs.timeout),
build.WithKustomizationFile(buildKsArgs.kustomizationFile),
build.WithDryRun(buildKsArgs.dryRun),
build.WithNamespace(*kubeconfigArgs.Namespace),
build.WithIgnore(buildKsArgs.ignorePaths),
build.WithStrictSubstitute(buildKsArgs.strictSubst),
build.WithRecursive(buildKsArgs.recursive),
build.WithLocalSources(buildKsArgs.localSources),
)
} else {
builder, err = build.NewBuilder(name, buildKsArgs.path,
build.WithClientConfig(kubeconfigArgs, kubeclientOptions), build.WithClientConfig(kubeconfigArgs, kubeclientOptions),
build.WithTimeout(rootArgs.timeout), build.WithTimeout(rootArgs.timeout),
build.WithKustomizationFile(buildKsArgs.kustomizationFile), build.WithKustomizationFile(buildKsArgs.kustomizationFile),
@ -133,8 +123,12 @@ func buildKsCmdRun(cmd *cobra.Command, args []string) (err error) {
build.WithRecursive(buildKsArgs.recursive), build.WithRecursive(buildKsArgs.recursive),
build.WithLocalSources(buildKsArgs.localSources), build.WithLocalSources(buildKsArgs.localSources),
) )
if buildKsArgs.dryRun {
options = append(options, build.WithDryRun(buildKsArgs.dryRun))
} }
builder, err = build.NewBuilder(name, buildKsArgs.path, options...)
if err != nil { if err != nil {
return err return err
} }

@ -99,23 +99,12 @@ func diffKsCmdRun(cmd *cobra.Command, args []string) error {
} }
var ( var (
options []build.BuilderOptionFunc
builder *build.Builder builder *build.Builder
err error err error
) )
if diffKsArgs.progressBar {
builder, err = build.NewBuilder(name, diffKsArgs.path, options = append(options,
build.WithClientConfig(kubeconfigArgs, kubeclientOptions),
build.WithTimeout(rootArgs.timeout),
build.WithKustomizationFile(diffKsArgs.kustomizationFile),
build.WithProgressBar(),
build.WithIgnore(diffKsArgs.ignorePaths),
build.WithStrictSubstitute(diffKsArgs.strictSubst),
build.WithRecursive(diffKsArgs.recursive),
build.WithLocalSources(diffKsArgs.localSources),
build.WithSingleKustomization(),
)
} else {
builder, err = build.NewBuilder(name, diffKsArgs.path,
build.WithClientConfig(kubeconfigArgs, kubeclientOptions), build.WithClientConfig(kubeconfigArgs, kubeclientOptions),
build.WithTimeout(rootArgs.timeout), build.WithTimeout(rootArgs.timeout),
build.WithKustomizationFile(diffKsArgs.kustomizationFile), build.WithKustomizationFile(diffKsArgs.kustomizationFile),
@ -125,8 +114,12 @@ func diffKsCmdRun(cmd *cobra.Command, args []string) error {
build.WithLocalSources(diffKsArgs.localSources), build.WithLocalSources(diffKsArgs.localSources),
build.WithSingleKustomization(), build.WithSingleKustomization(),
) )
if diffKsArgs.progressBar {
options = append(options, build.WithProgressBar())
} }
builder, err = build.NewBuilder(name, diffKsArgs.path, options...)
if err != nil { if err != nil {
return &RequestError{StatusCode: 2, Err: err} return &RequestError{StatusCode: 2, Err: err}
} }

Loading…
Cancel
Save