From 8db30d686a5298a28c6275cc05fb24fb1cfa1ba5 Mon Sep 17 00:00:00 2001 From: Raman Tenneti Date: Tue, 6 Jul 2021 21:30:06 -0700 Subject: [PATCH] superproject - More friendly user message when superproject failed. superproject is going to be default for some users. This change doesn't fail for repo init or repo sync if source couldn't be synced because of errors in superproject and superproject=true in the config file. The commands will fail if --use-superproject is specified on the command line explicitly. The error messages are logged with trace2 event logs and will be monitored. + sync - When there are errors with superproject and git_superproject says it is fatal failure, sync will exit only when --use-superproject option is specified on the command line. + init - command doesn't fail *if there are any superproject errors), but it will print a warning message and logs message via trace2 event logs. For fatal errors, init will exit only when --use-superproject option is specified on the command line. + All git commands log the command that is being executed so trace2 event logs will know the manifest, remote url and the branch name. There is no functional change other than fatal errors are honored with --use-supeproject option with init/sync commands. Tested the code with the following commands. $ ./run_tests -v Test 1 - sync'ing without errors -------------------------------- Added the following lines to '~/.repoconfig/config [repo] superproject = true $ repo_dev init -u https://android.googlesource.com/platform/manifest -b android-s-beta-2 NOTICE: --use-superproject is in beta; report any issues to the address described in `repo version` repo: error: git fetch call failed, command: git ['fetch', 'https://android.googlesource.com/platform/superproject', '--depth', '1', '--force', '--no-tags', '--filter', 'blob:none', 'android-s-beta-2:android-s-beta-2'], return code: 128, stderr: fatal: couldn't find remote ref android-s-beta-2 warning: git update of superproject failed, repo sync will not use superproject to fetch source; while this error is not fatal, and you can continue to run repo sync, please run repo init with the --no-use-superproject option to stop seeing this warning Your identity is: Raman Tenneti If you want to change this, please re-run 'repo init' with --config-name repo has been initialized in /usr/local/google/home/rtenneti/work/drive2/android/test $ repo_dev sync remote: Total 4 (delta 1), reused 4 (delta 1) NOTICE: --use-superproject is in beta; report any issues to the address described in `repo version` /usr/local/google/home/rtenneti/work/drive2/android/aosp/.repo/exp-superproject/925043f706ba64db713e9bf3b55987e2-superproject.git: Initial setup for superproject completed. ... Test 2 - init and sync fail when --use-superproject option is passed -------------------------------------------------------------------- $ repo_dev init -u https://android.googlesource.com/platform/manifest -b android-s-beta-2 --use-superproject remote: Total 57 (delta 16), reused 56 (delta 16) NOTICE: --use-superproject is in beta; report any issues to the address described in `repo version` repo: error: git fetch call failed, command: git ['fetch', 'https://android.googlesource.com/platform/superproject', '--depth', '1', '--force', '--no-tags', '--filter', 'blob:none', 'android-s-beta-2:android-s-beta-2'], return code: 128, stderr: fatal: couldn't find remote ref android-s-beta-2 warning: git update of superproject failed, repo sync will not use superproject to fetch source; while this error is not fatal, and you can continue to run repo sync, please run repo init with the --no-use-superproject option to stop seeing this warning rtenneti@rtenneti2:~/work/drive2/android/test$ repo_dev sync --use-superproject NOTICE: --use-superproject is in beta; report any issues to the address described in `repo version` repo: error: git fetch call failed, command: git ['fetch', 'https://android.googlesource.com/platform/superproject', '--depth', '1', '--force', '--no-tags', '--filter', 'blob:none', 'android-s-beta-2:android-s-beta-2'], return code: 128, stderr: fatal: couldn't find remote ref android-s-beta-2 warning: Cannot get project commit ids from manifest warning: Update of revisionId from superproject has failed, repo sync will not use superproject to fetch the source. Please resync with the --no-use-superproject option to avoid this repo warning. Test 3 - git fetch command fails and git command is printed ----------------------------------------------------------- With config change $ repo_dev init -u https://android.googlesource.com/platform/manifest -b android-s-beta-2 ... NOTICE: --use-superproject is in beta; report any issues to the address described in `repo version` .../android/test/.repo/exp-superproject/925043f706ba64db713e9bf3b55987e2-superproject.git: Performing initial setup for superproject; this might take several minutes. repo: error: git fetch call failed,command: git ['fetch', 'https://android.googlesource.com/platform/superproject', '--depth', '1', '--force', '--no-tags', '--filter', 'blob:none', 'android-s-beta-2:android-s-beta-2'], return code: 128, stderr: fatal: couldn't find remote ref android-s-beta-2 warning: git update of superproject failed, repo sync will not use superproject to fetch source; while this error is not fatal and you can continue to run repo sync please run repo init with the --no-use-superproject option to avoid the repo warning Your identity is: Raman Tenneti If you want to change this, please re-run 'repo init' with --config-name repo has been initialized in .... Test 4 - no superproject tag ----------------------------- $ repo_dev init -u https://android.googlesource.com/platform/manifest -b pie-dev ... NOTICE: --use-superproject is in beta; report any issues to the address described in `repo version` repo error: superproject tag is not defined in manifest: .../android/pie_dev/.repo/manifest.xml warning: git update of superproject failed, repo sync will not use superproject to fetch source; while this error is not fatal and you can continue to run repo sync please run repo init with the --no-use-superproject option to avoid the repo warning Your identity is: Raman Tenneti If you want to change this, please re-run 'repo init' with --config-name repo has been initialized in ... $ repo_dev sync NOTICE: --use-superproject is in beta; report any issues to the address described in `repo version` repo error: superproject tag is not defined in manifest: /usr/local/google/home/rtenneti/work/drive2/android/pie_dev/.repo/manifest.xml warning: Cannot get project commit ids from manifest warning: Update of revsionId from superproject has failed. Please resync with --no-use-superproject option to avoid the repo warning. Bug: [google internal] b/192614798 Bug: [google internal] b/Bug: [google internal] b/192614798 Change-Id: I9a97a0e7d9e609fad151bd7dd9cfc523eaa887cd Reviewed-on: https://gerrit-review.googlesource.com/c/git-repo/+/311502 Reviewed-by: Amith Dsouza Reviewed-by: Xin Li Tested-by: Raman Tenneti --- git_superproject.py | 51 ++++++++++++++++++++------------------------- subcmds/init.py | 8 +++++-- subcmds/sync.py | 7 ++++--- 3 files changed, 33 insertions(+), 33 deletions(-) diff --git a/git_superproject.py b/git_superproject.py index 2a47847a..86100960 100644 --- a/git_superproject.py +++ b/git_superproject.py @@ -117,6 +117,11 @@ class Superproject(object): branch = branch[len(R_HEADS):] return branch + def _LogError(self, message): + """Logs message to stderr and _git_event_log.""" + print(message, file=sys.stderr) + self._git_event_log.ErrorEvent(message, '') + def _Init(self): """Sets up a local Git repository to get a copy of a superproject. @@ -136,8 +141,8 @@ class Superproject(object): capture_stderr=True) retval = p.Wait() if retval: - print('repo: error: git init call failed with return code: %r, stderr: %r' % - (retval, p.stderr), file=sys.stderr) + self._LogError(f'repo: error: git init call failed, command: git {cmd}, ' + f'return code: {retval}, stderr: {p.stderr}') return False return True @@ -151,8 +156,7 @@ class Superproject(object): True if fetch is successful, or False. """ if not os.path.exists(self._work_git): - print('git fetch missing drectory: %s' % self._work_git, - file=sys.stderr) + self._LogError(f'git fetch missing directory: {self._work_git}') return False if not git_require((2, 28, 0)): print('superproject requires a git version 2.28 or later', file=sys.stderr) @@ -167,8 +171,8 @@ class Superproject(object): capture_stderr=True) retval = p.Wait() if retval: - print('repo: error: git fetch call failed with return code: %r, stderr: %r' % - (retval, p.stderr), file=sys.stderr) + self._LogError(f'repo: error: git fetch call failed, command: git {cmd}, ' + f'return code: {retval}, stderr: {p.stderr}') return False return True @@ -181,8 +185,7 @@ class Superproject(object): data: data returned from 'git ls-tree ...' instead of None. """ if not os.path.exists(self._work_git): - print('git ls-tree missing drectory: %s' % self._work_git, - file=sys.stderr) + self._LogError(f'git ls-tree missing directory: {self._work_git}') return None data = None branch = 'HEAD' if not self._branch else self._branch @@ -197,8 +200,8 @@ class Superproject(object): if retval == 0: data = p.stdout else: - print('repo: error: git ls-tree call failed with return code: %r, stderr: %r' % ( - retval, p.stderr), file=sys.stderr) + self._LogError(f'repo: error: git ls-tree call failed, command: git {cmd}, ' + f'return code: {retval}, stderr: {p.stderr}') return data def Sync(self): @@ -211,17 +214,15 @@ class Superproject(object): 'address described in `repo version`', file=sys.stderr) if not self._manifest.superproject: - msg = (f'repo error: superproject tag is not defined in manifest: ' - f'{self._manifest.manifestFile}') - print(msg, file=sys.stderr) - self._git_event_log.ErrorEvent(msg, '') + self._LogError(f'repo error: superproject tag is not defined in manifest: ' + f'{self._manifest.manifestFile}') return SyncResult(False, False) should_exit = True url = self._manifest.superproject['remote'].url if not url: - print('error: superproject URL is not defined in manifest', - file=sys.stderr) + self._LogError(f'repo error: superproject URL is not defined in manifest: ' + f'{self._manifest.manifestFile}') return SyncResult(False, should_exit) if not self._Init(): @@ -244,7 +245,7 @@ class Superproject(object): data = self._LsTree() if not data: - print('error: git ls-tree failed to return data for superproject', + print('warning: git ls-tree failed to return data for superproject', file=sys.stderr) return CommitIdsResult(None, True) @@ -271,9 +272,7 @@ class Superproject(object): manifest_path: Path name of the file into which manifest is written instead of None. """ if not os.path.exists(self._superproject_path): - print('error: missing superproject directory %s' % - self._superproject_path, - file=sys.stderr) + self._LogError(f'error: missing superproject directory: {self._superproject_path}') return None manifest_str = self._manifest.ToXml(groups=self._manifest.GetGroupsStr()).toxml() manifest_path = self._manifest_path @@ -281,9 +280,7 @@ class Superproject(object): with open(manifest_path, 'w', encoding='utf-8') as fp: fp.write(manifest_str) except IOError as e: - print('error: cannot write manifest to %s:\n%s' - % (manifest_path, e), - file=sys.stderr) + self._LogError(f'error: cannot write manifest to : {manifest_path} {e}') return None return manifest_path @@ -319,7 +316,7 @@ class Superproject(object): commit_ids_result = self._GetAllProjectsCommitIds() commit_ids = commit_ids_result.commit_ids if not commit_ids: - print('error: Cannot get project commit ids from manifest', file=sys.stderr) + print('warning: Cannot get project commit ids from manifest', file=sys.stderr) return UpdateProjectsResult(None, commit_ids_result.fatal) projects_missing_commit_ids = [] @@ -334,10 +331,8 @@ class Superproject(object): # If superproject doesn't have a commit id for a project, then report an # error event and continue as if do not use superproject is specified. if projects_missing_commit_ids: - msg = (f'error: please file a bug using {self._manifest.contactinfo.bugurl} ' - f'to report missing commit_ids for: {projects_missing_commit_ids}') - print(msg, file=sys.stderr) - self._git_event_log.ErrorEvent(msg, '') + self._LogError(f'error: please file a bug using {self._manifest.contactinfo.bugurl} ' + f'to report missing commit_ids for: {projects_missing_commit_ids}') return UpdateProjectsResult(None, False) for project in projects: diff --git a/subcmds/init.py b/subcmds/init.py index 536e367c..5671fc24 100644 --- a/subcmds/init.py +++ b/subcmds/init.py @@ -100,8 +100,12 @@ to update the working directory files. quiet=opt.quiet) sync_result = superproject.Sync() if not sync_result.success: - print('error: git update of superproject failed', file=sys.stderr) - if sync_result.fatal: + print('warning: git update of superproject failed, repo sync will not ' + 'use superproject to fetch source; while this error is not fatal, ' + 'and you can continue to run repo sync, please run repo init with ' + 'the --no-use-superproject option to stop seeing this warning', + file=sys.stderr) + if sync_result.fatal and opt.use_superproject is not None: sys.exit(1) def _SyncManifest(self, opt): diff --git a/subcmds/sync.py b/subcmds/sync.py index bd5ffd0c..e6dbf1c8 100644 --- a/subcmds/sync.py +++ b/subcmds/sync.py @@ -309,10 +309,11 @@ later is required to fix a server side protocol bug. if manifest_path: self._ReloadManifest(manifest_path, load_local_manifests) else: - print('error: Update of revsionId from superproject has failed. ' - 'Please resync with --no-use-superproject option', + print('warning: Update of revisionId from superproject has failed, ' + 'repo sync will not use superproject to fetch the source. ', + 'Please resync with the --no-use-superproject option to avoid this repo warning.', file=sys.stderr) - if update_result.fatal: + if update_result.fatal and opt.use_superproject is not None: sys.exit(1) return manifest_path