Merge pull request #1472 from jenkins-x/sparse2

fix: Reuse sparse checkout

OWNERS

@@ -1,38 +1,4 @@
 approvers:
-- abayer
-- cagiti
-- ccojocar
-- daveconde
-- dgozalo
-- eddycharly
-- garethjevans
-- hferentschik
-- jstrachan
-- macox
-- mrageh
-- pmuir
-- rawlingsj
-- warrenbailey
-- wbrefvem
-- ankitm123
-- tomhobson
-- babadofar
+- maintainers
 reviewers:
-- abayer
-- cagiti
-- ccojocar
-- daveconde
-- dgozalo
-- eddycharly
-- garethjevans
-- hferentschik
-- jstrachan
-- macox
-- mrageh
-- pmuir
-- rawlingsj
-- warrenbailey
-- wbrefvem
-- ankitm123
-- tomhobson
-- babadofar
+- maintainers

OWNERS_ALIASES

@@ -0,0 +1,3 @@
+foreignAliases:
+- name: jx-community
+  org: jenkins-x

pkg/git/git.go

@@ -206,7 +206,7 @@ func (c *client) SparseClone(repo string, sparseCheckoutPatterns []string) (*Rep
 		}
 		remote := getRemote(repo, c, base)
 
-		if b, err := retryCmd(c.logger, "", c.git, "clone", "--no-checkout", "--filter=blob:none", "--sparse", "--depth=1", remote, cache); err != nil {
+		if b, err := retryCmd(c.logger, "", c.git, "clone", "--no-checkout", "--filter=blob:none", "--sparse", "--depth=10", "--no-single-branch", remote, cache); err != nil {
 			return nil, fmt.Errorf("failed to clone repository: %v. output: %s", err, string(b))
 		}
 		if b, err := retryCmd(c.logger, cache, c.git, append([]string{"sparse-checkout", "set"}, sparseCheckoutPatterns...)...); err != nil {
@@ -217,7 +217,7 @@ func (c *client) SparseClone(repo string, sparseCheckoutPatterns []string) (*Rep
 	} else {
 		// Cache hit. Do a git fetch to keep updated.
 		c.logger.Infof("Fetching %s.", repo)
-		if b, err := retryCmd(c.logger, cache, c.git, "fetch", "--depth=1"); err != nil {
+		if b, err := retryCmd(c.logger, cache, c.git, "fetch", "--depth=10"); err != nil {
 			return nil, fmt.Errorf("git fetch error: %v. output: %s", err, string(b))
 		}
 	}

pkg/git/v2/interactor.go

@@ -116,7 +116,7 @@ func (i *interactor) Clean() error {
 func (i *interactor) Clone(repo string) error {
 	sparseCheckout, _ := strconv.ParseBool(os.Getenv("SPARSE_CHECKOUT"))
 	i.logger.Debugf("Creating a clone of the repo at %s from %s", i.dir, repo)
-	args := []string{"clone", "--no-checkout", "--depth=1"}
+	args := []string{"clone", "--no-checkout", "--depth=10", "--no-single-branch"}
 	if sparseCheckout {
 		args = append(args, "--filter=blob:none")
 	}
@@ -143,7 +143,7 @@ func (i *interactor) MirrorClone() error {
 	if err != nil {
 		return fmt.Errorf("could not resolve remote for cloning: %v", err)
 	}
-	out, err := i.executor.Run("clone", "--mirror", "--depth=1", remote, i.dir)
+	out, err := i.executor.Run("clone", "--mirror", "--depth=10", "--no-single-branch", remote, i.dir)
 	if err != nil {
 		// the returned error is not being reported
 		i.logger.Errorf("error creating a mirror clone: %v %v", err, string(out))

pkg/git/v2/interactor_test.go

@@ -42,12 +42,12 @@ func TestInteractor_Clone(t *testing.T) {
 			dir:  "/else",
 			from: "/somewhere",
 			responses: map[string]execResponse{
-				"clone --no-checkout --depth=1 /somewhere /else": {
+				"clone --no-checkout --depth=10 --no-single-branch /somewhere /else": {
 					out: []byte(`ok`),
 				},
 			},
 			expectedCalls: [][]string{
-				{"clone", "--no-checkout", "--depth=1", "/somewhere", "/else"},
+				{"clone", "--no-checkout", "--depth=10", "--no-single-branch", "/somewhere", "/else"},
 			},
 			expectedErr: false,
 		},
@@ -56,12 +56,12 @@ func TestInteractor_Clone(t *testing.T) {
 			dir:  "/else",
 			from: "/somewhere",
 			responses: map[string]execResponse{
-				"clone --no-checkout --depth=1 /somewhere /else": {
+				"clone --no-checkout --depth=10 --no-single-branch /somewhere /else": {
 					err: errors.New("oops"),
 				},
 			},
 			expectedCalls: [][]string{
-				{"clone", "--no-checkout", "--depth=1", "/somewhere", "/else"},
+				{"clone", "--no-checkout", "--depth=10", "--no-single-branch", "/somewhere", "/else"},
 			},
 			expectedErr: true,
 		},
@@ -109,12 +109,12 @@ func TestInteractor_MirrorClone(t *testing.T) {
 				return "someone.com", nil
 			},
 			responses: map[string]execResponse{
-				"clone --mirror --depth=1 someone.com /else": {
+				"clone --mirror --depth=10 --no-single-branch someone.com /else": {
 					out: []byte(`ok`),
 				},
 			},
 			expectedCalls: [][]string{
-				{"clone", "--mirror", "--depth=1", "someone.com", "/else"},
+				{"clone", "--mirror", "--depth=10", "--no-single-branch", "someone.com", "/else"},
 			},
 			expectedErr: false,
 		},
@@ -135,12 +135,12 @@ func TestInteractor_MirrorClone(t *testing.T) {
 				return "someone.com", nil
 			},
 			responses: map[string]execResponse{
-				"clone --mirror --depth=1 someone.com /else": {
+				"clone --mirror --depth=10 --no-single-branch someone.com /else": {
 					err: errors.New("oops"),
 				},
 			},
 			expectedCalls: [][]string{
-				{"clone", "--mirror", "--depth=1", "someone.com", "/else"},
+				{"clone", "--mirror", "--depth=10", "--no-single-branch", "someone.com", "/else"},
 			},
 			expectedErr: true,
 		},