diff options
author | Josip Sokcevic <sokcevic@chromium.org> | 2025-01-31 12:00:52 -0800 |
---|---|---|
committer | LUCI <gerrit-scoped@luci-project-accounts.iam.gserviceaccount.com> | 2025-01-31 12:01:49 -0800 |
commit | 5ae8292fea94bc26aa8c7345f5230899d3d45a63 (patch) | |
tree | 16d08791b71be014af4927fc9da69af5365031fc /subcmds/sync.py | |
parent | dfdf577e98f6e9b13a3236767316863b3a995c01 (diff) | |
download | git-repo-5ae8292fea94bc26aa8c7345f5230899d3d45a63.tar.gz |
Revert "sync: skip network half on repo upgrade"
This reverts commit 61224d01fa29bcf54dd6d521e09e09a8c0da77fe.
Reason for revert: the manifest will be updated during in the
post-upgrade process, and that can result in a missing object in
LocalHalf, since NetworkHalf is not skipped.
Bug: b/392979411
Change-Id: I8a46e5b54093ed78285c8b30f000bb08a8244179
Reviewed-on: https://gerrit-review.googlesource.com/c/git-repo/+/450181
Tested-by: Josip Sokcevic <sokcevic@chromium.org>
Commit-Queue: Josip Sokcevic <sokcevic@chromium.org>
Reviewed-by: Scott Lee <ddoman@google.com>
Diffstat (limited to 'subcmds/sync.py')
-rw-r--r-- | subcmds/sync.py | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/subcmds/sync.py b/subcmds/sync.py index f7ed49e4..978b3beb 100644 --- a/subcmds/sync.py +++ b/subcmds/sync.py | |||
@@ -1834,7 +1834,7 @@ later is required to fix a server side protocol bug. | |||
1834 | 1834 | ||
1835 | self._fetch_times = _FetchTimes(manifest) | 1835 | self._fetch_times = _FetchTimes(manifest) |
1836 | self._local_sync_state = LocalSyncState(manifest) | 1836 | self._local_sync_state = LocalSyncState(manifest) |
1837 | if not opt.local_only and not opt.repo_upgraded: | 1837 | if not opt.local_only: |
1838 | with multiprocessing.Manager() as manager: | 1838 | with multiprocessing.Manager() as manager: |
1839 | with ssh.ProxyManager(manager) as ssh_proxy: | 1839 | with ssh.ProxyManager(manager) as ssh_proxy: |
1840 | # Initialize the socket dir once in the parent. | 1840 | # Initialize the socket dir once in the parent. |