Add more clone logging (#4593)

This commit adds timing information to the logs when clones complete successfully. There are several other more robust but also more involved interventions we could do, and I think we should do them, but this is a strict improvement that's easy to get out as a first step.

I also added logging at the precise beginning of the clone command so we know when we're doing that.

Author: rosecodym

pkg/sources/git/git.go

@@ -490,6 +490,8 @@ func CloneRepo(ctx context.Context, userInfo *url.Userinfo, gitURL string, clone
 // executeClone prepares the Git URL, constructs, and executes the git clone command using the provided
 // clonePath. It then opens the cloned repository, returning a git.Repository object.
 func executeClone(ctx context.Context, params cloneParams) (*git.Repository, error) {
+	start := time.Now()
+
 	cloneURL, err := GitURLParse(params.gitURL)
 	if err != nil {
 		return nil, err
@@ -543,6 +545,7 @@ func executeClone(ctx context.Context, params cloneParams) (*git.Repository, err
 		"args", params.args,
 	)
 
+	logger.V(3).Info("executing git clone command")
 	outputBytes, err := cloneCmd.CombinedOutput()
 	var output string
 	if secretForRedaction != "" {
@@ -570,7 +573,7 @@ func executeClone(ctx context.Context, params cloneParams) (*git.Repository, err
 	if err != nil {
 		return nil, fmt.Errorf("could not open cloned repo: %w", err)
 	}
-	logger.V(1).Info("successfully cloned repo")
+	logger.V(1).Info("successfully cloned repo", "time_seconds", time.Since(start).Seconds())
 
 	metricsInstance.RecordCloneOperation(statusSuccess, cloneSuccess, 0)