commit d379e77f44d11da1ab817f3796f9559f163c1b0f
author Yiwei Zhang <yiwzhang@google.com> Wed Dec 20 20:39:59 2023 +0000
committer LUCI <gerrit-scoped@luci-project-accounts.iam.gserviceaccount.com> Fri Feb 02 18:35:13 2024 +0000
tree f266bea2e40205a9ce97106c74556cceda5b191f
parent 4217a82bec7b95c1a0bc7b081a1764a6a6d6c59b

stop passing project to UpdateManifestError

UpdateManifestError inherits from RepoExitError which inherits
from BaseRepoError. None of them takes project as kwargs
causing the error like "UpdateManifestError() takes no keyword
arguments" in b/317183321

[1]: https://gerrit.googlesource.com/git-repo/+/449b23b698d7d4b13909667a49a0698eb495eeaa/error.py#144

Bug: b/317183321
Change-Id: I64c3dc502027f9dda56a0824f2712364b4502934
Reviewed-on: https://gerrit-review.googlesource.com/c/git-repo/+/398997
Commit-Queue: Yiwei Zhang <yiwzhang@google.com>
Tested-by: Yiwei Zhang <yiwzhang@google.com>
Reviewed-by: Josip Sokcevic <sokcevic@google.com>
Reviewed-by: Jason Chang <jasonnc@google.com>

subcmds/sync.py

diff --git a/subcmds/sync.py b/subcmds/sync.py
index ec4bf60..233c9f4 100644
--- a/subcmds/sync.py
+++ b/subcmds/sync.py
@@ -1544,9 +1544,7 @@
                 mp, event_log.TASK_SYNC_LOCAL, start, time.time(), clean
             )
             if not clean:
-                raise UpdateManifestError(
-                    aggregate_errors=errors, project=mp.name
-                )
+                raise UpdateManifestError(aggregate_errors=errors)
             self._ReloadManifest(manifest_name, mp.manifest)
 
     def ValidateOptions(self, opt, args):