Revert "scm: use dulwich backend when fetching exps during clone/pull" (#9041)

Revert "scm: use dulwich backend when fetching exps during clone/pull (#9023)"

This reverts commit ed1ab7fa75.
This commit is contained in:
skshetry
2023-02-16 18:07:45 +05:45
committed by GitHub
parent 3f0da8f6cc
commit d7ea7c358d
3 changed files with 9 additions and 8 deletions
+1 -1
View File
@@ -253,7 +253,7 @@ def _pull(git: "Git", unshallow: bool = False):
git.fetch(unshallow=unshallow)
_merge_upstream(git)
fetch_all_exps(git, "origin", backends=["dulwich"])
fetch_all_exps(git, "origin")
def _merge_upstream(git: "Git"):
+7 -6
View File
@@ -5,7 +5,6 @@ from collections import defaultdict
from functools import wraps
from typing import (
TYPE_CHECKING,
Any,
Callable,
Dict,
Generator,
@@ -280,15 +279,17 @@ def check_ref_format(scm: "Git", ref: ExpRefInfo):
)
def fetch_all_exps(
scm: "Git", url: str, progress: Optional[Callable] = None, **kwargs: Any
):
def fetch_all_exps(scm: "Git", url: str, progress: Optional[Callable] = None):
refspecs = [
f"{ref}:{ref}"
for ref in iter_remote_refs(scm, url, base=EXPS_NAMESPACE, **kwargs)
for ref in iter_remote_refs(scm, url, base=EXPS_NAMESPACE)
if not (ref.startswith(EXEC_NAMESPACE) or ref in STASHES)
]
scm.fetch_refspecs(url, refspecs, progress=progress, **kwargs)
scm.fetch_refspecs(
url,
refspecs,
progress=progress,
)
def get_random_exp_name(scm, baseline_rev):
+1 -1
View File
@@ -179,7 +179,7 @@ def clone(url: str, to_path: str, **kwargs):
try:
git = Git.clone(url, to_path, progress=pbar.update_git, **kwargs)
if "shallow_branch" not in kwargs:
fetch_all_exps(git, url, progress=pbar.update_git, backends=["dulwich"])
fetch_all_exps(git, url, progress=pbar.update_git)
return git
except InternalCloneError as exc:
raise CloneError("SCM error") from exc