Remove GitImplementation default

As the field in the CRD is optional.

Signed-off-by: Hidde Beydals <hello@hidde.co>
pull/721/head
Hidde Beydals 4 years ago
parent 6396b25886
commit a260403334

@ -164,14 +164,13 @@ func applyInstallManifests(ctx context.Context, manifestPath string, components
func generateSyncManifests(url, branch, name, namespace, targetPath, tmpDir string, interval time.Duration) (string, error) { func generateSyncManifests(url, branch, name, namespace, targetPath, tmpDir string, interval time.Duration) (string, error) {
opts := sync.Options{ opts := sync.Options{
Name: name, Name: name,
Namespace: namespace, Namespace: namespace,
URL: url, URL: url,
Branch: branch, Branch: branch,
Interval: interval, Interval: interval,
TargetPath: targetPath, TargetPath: targetPath,
ManifestFile: sync.MakeDefaultOptions().ManifestFile, ManifestFile: sync.MakeDefaultOptions().ManifestFile,
GitImplementation: sync.MakeDefaultOptions().GitImplementation,
} }
manifest, err := sync.Generate(opts) manifest, err := sync.Generate(opts)

@ -18,8 +18,6 @@ package sync
import ( import (
"time" "time"
sourcev1 "github.com/fluxcd/source-controller/api/v1beta1"
) )
type Options struct { type Options struct {
@ -42,6 +40,6 @@ func MakeDefaultOptions() Options {
Branch: "main", Branch: "main",
ManifestFile: "gotk-sync.yaml", ManifestFile: "gotk-sync.yaml",
TargetPath: "", TargetPath: "",
GitImplementation: sourcev1.GoGitImplementation, GitImplementation: "",
} }
} }

Loading…
Cancel
Save