fix(ci_visibility): do not fetch tags while unshallowing git repo [backport 3.17] (#15139)

Backport aeb5df4 from #15133 to 3.17.

## Description

Currently when calling `git fetch` to unshallow a repo, we end up
fetching repo tags, even though we don't need them. This can have a
performance impact in repos with a large number of tags. This PR changes
it to call `git fetch` with `--no-tags`.

## Testing

Unit tests and manual testing.

## Risks

None.

## Additional Notes

None.

Co-authored-by: Vítor De Araújo <[email protected]>
Co-authored-by: Christophe Papazian <[email protected]>

Author: 3 people

ddtrace/ext/git.py

@@ -206,6 +206,7 @@ def _unshallow_repository_with_details(
         "--update-shallow",
         "--filter=blob:none",
         "--recurse-submodules=no",
+        "--no-tags",
     ]
     if repo is not None:
         cmd.append(repo)

releasenotes/notes/ci_visibility-fix-unshallow-no-tags-bbe7e26aadcf250f.yaml

@@ -0,0 +1,5 @@
+---
+fixes:
+  - |
+    CI Visibility: this fix resolves an issue where repo tags would be fetched while unshallowing to extract commit
+    metadata, causing performance issues for repos with a large number of tags.

tests/ci_visibility/test_ci_visibility.py

@@ -1023,6 +1023,7 @@ def test_unshallow_repository_local_head():
                     "--update-shallow",
                     "--filter=blob:none",
                     "--recurse-submodules=no",
+                    "--no-tags",
                     "origin",
                     "myfakesha",
                     cwd="/path/to/repo",
@@ -1051,6 +1052,7 @@ def test_unshallow_repository_upstream():
                         "--update-shallow",
                         "--filter=blob:none",
                         "--recurse-submodules=no",
+                        "--no-tags",
                         "origin",
                         "myupstreamsha",
                         cwd="/path/to/repo",
@@ -1080,6 +1082,7 @@ def test_unshallow_repository_full():
                         "--update-shallow",
                         "--filter=blob:none",
                         "--recurse-submodules=no",
+                        "--no-tags",
                         "origin",
                         cwd="/path/to/repo",
                     )

tests/tracer/test_ci.py

@@ -369,6 +369,7 @@ def test_unshallow_repository_bare(git_repo):
             "--update-shallow",
             "--filter=blob:none",
             "--recurse-submodules=no",
+            "--no-tags",
             cwd=git_repo,
         )
 
@@ -384,6 +385,7 @@ def test_unshallow_repository_bare_repo(git_repo):
             "--update-shallow",
             "--filter=blob:none",
             "--recurse-submodules=no",
+            "--no-tags",
             "myremote",
             cwd=git_repo,
         )
@@ -400,6 +402,7 @@ def test_unshallow_repository_bare_repo_refspec(git_repo):
             "--update-shallow",
             "--filter=blob:none",
             "--recurse-submodules=no",
+            "--no-tags",
             "myremote",
             "mycommitshaaaaaaaaaaaa123",
             cwd=git_repo,