api/v1: Remove deprecated `GitImplementation`

Signed-off-by: Hidde Beydals <hidde@hhh.computer>

api/v1/gitrepository_types.go

@@ -94,15 +94,6 @@ type GitRepositorySpec struct {
 	// +optional
 	Suspend bool `json:"suspend,omitempty"`
 
-	// GitImplementation specifies which Git client library implementation to
-	// use. Defaults to 'go-git', valid values are ('go-git', 'libgit2').
-	// Deprecated: gitImplementation is deprecated now that 'go-git' is the
-	// only supported implementation.
-	// +kubebuilder:validation:Enum=go-git;libgit2
-	// +kubebuilder:default:=go-git
-	// +optional
-	GitImplementation string `json:"gitImplementation,omitempty"`
-
 	// RecurseSubmodules enables the initialization of all submodules within
 	// the GitRepository as cloned from the URL, using their default settings.
 	// +optional

config/crd/bases/source.toolkit.fluxcd.io_gitrepositories.yaml

@@ -51,16 +51,6 @@ spec:
             description: GitRepositorySpec specifies the required configuration to
               produce an Artifact for a Git repository.
             properties:
-              gitImplementation:
-                default: go-git
-                description: 'GitImplementation specifies which Git client library
-                  implementation to use. Defaults to ''go-git'', valid values are
-                  (''go-git'', ''libgit2''). Deprecated: gitImplementation is deprecated
-                  now that ''go-git'' is the only supported implementation.'
-                enum:
-                - go-git
-                - libgit2
-                type: string
               ignore:
                 description: Ignore overrides the set of excluded patterns in the
                   .sourceignore format (which is the same as .gitignore). If not provided,

controllers/gitrepository_controller_test.go

@@ -1740,10 +1740,9 @@ func TestGitRepositoryReconciler_ConditionsUpdate(t *testing.T) {
 					Finalizers: []string{sourcev1.SourceFinalizer},
 				},
 				Spec: sourcev1.GitRepositorySpec{
-					URL:               server.HTTPAddress() + repoPath,
+					URL:      server.HTTPAddress() + repoPath,
-					GitImplementation: sourcev1.GoGitImplementation,
+					Interval: metav1.Duration{Duration: interval},
-					Interval:          metav1.Duration{Duration: interval},
+					Timeout:  &metav1.Duration{Duration: timeout},
-					Timeout:           &metav1.Duration{Duration: timeout},
 				},
 			}
 

docs/api/v1/source.md

@@ -184,21 +184,6 @@ GitRepository.</p>
 </tr>
 <tr>
 <td>
-<code>gitImplementation</code><br>
-<em>
-string
-</em>
-</td>
-<td>
-<em>(Optional)</em>
-<p>GitImplementation specifies which Git client library implementation to
-use. Defaults to &lsquo;go-git&rsquo;, valid values are (&lsquo;go-git&rsquo;, &lsquo;libgit2&rsquo;).
-Deprecated: gitImplementation is deprecated now that &lsquo;go-git&rsquo; is the
-only supported implementation.</p>
-</td>
-</tr>
-<tr>
-<td>
 <code>recurseSubmodules</code><br>
 <em>
 bool
@@ -635,21 +620,6 @@ GitRepository.</p>
 </tr>
 <tr>
 <td>
-<code>gitImplementation</code><br>
-<em>
-string
-</em>
-</td>
-<td>
-<em>(Optional)</em>
-<p>GitImplementation specifies which Git client library implementation to
-use. Defaults to &lsquo;go-git&rsquo;, valid values are (&lsquo;go-git&rsquo;, &lsquo;libgit2&rsquo;).
-Deprecated: gitImplementation is deprecated now that &lsquo;go-git&rsquo; is the
-only supported implementation.</p>
-</td>
-</tr>
-<tr>
-<td>
 <code>recurseSubmodules</code><br>
 <em>
 bool