diff options
| author | Alex Bennée <alex.bennee@linaro.org> | 2024-10-23 12:33:58 +0100 |
|---|---|---|
| committer | Alex Bennée <alex.bennee@linaro.org> | 2024-10-24 09:56:29 +0100 |
| commit | 97f116f9c6fd127b6ed2953993fa9fb05e82f450 (patch) | |
| tree | cc89bc33dbdd748836cf2cc969c4144c180470be | |
| parent | 24be5341fbeea341cca38b59d4c0928a8cf5fac1 (diff) | |
| download | qemu-97f116f9c6fd127b6ed2953993fa9fb05e82f450.zip qemu-97f116f9c6fd127b6ed2953993fa9fb05e82f450.tar.gz qemu-97f116f9c6fd127b6ed2953993fa9fb05e82f450.tar.bz2 | |
gitlab: make check-[dco|patch] a little more verbose
When git fails the rather terse backtrace only indicates it failed
without some useful context. Add some to make the log a little more
useful.
Reviewed-by: Daniel P. Berrangé <berrange@redhat.com>
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Message-Id: <20241023113406.1284676-11-alex.bennee@linaro.org>
| -rwxr-xr-x | .gitlab-ci.d/check-dco.py | 5 | ||||
| -rwxr-xr-x | .gitlab-ci.d/check-patch.py | 5 |
2 files changed, 4 insertions, 6 deletions
diff --git a/.gitlab-ci.d/check-dco.py b/.gitlab-ci.d/check-dco.py index 632c8bc..d221b16 100755 --- a/.gitlab-ci.d/check-dco.py +++ b/.gitlab-ci.d/check-dco.py @@ -19,10 +19,9 @@ cwd = os.getcwd() reponame = os.path.basename(cwd) repourl = "https://gitlab.com/%s/%s.git" % (namespace, reponame) +print(f"adding upstream git repo @ {repourl}") subprocess.check_call(["git", "remote", "add", "check-dco", repourl]) -subprocess.check_call(["git", "fetch", "check-dco", "master"], - stdout=subprocess.DEVNULL, - stderr=subprocess.DEVNULL) +subprocess.check_call(["git", "fetch", "check-dco", "master"]) ancestor = subprocess.check_output(["git", "merge-base", "check-dco/master", "HEAD"], diff --git a/.gitlab-ci.d/check-patch.py b/.gitlab-ci.d/check-patch.py index 39e2b40..68c549a 100755 --- a/.gitlab-ci.d/check-patch.py +++ b/.gitlab-ci.d/check-patch.py @@ -19,13 +19,12 @@ cwd = os.getcwd() reponame = os.path.basename(cwd) repourl = "https://gitlab.com/%s/%s.git" % (namespace, reponame) +print(f"adding upstream git repo @ {repourl}") # GitLab CI environment does not give us any direct info about the # base for the user's branch. We thus need to figure out a common # ancestor between the user's branch and current git master. subprocess.check_call(["git", "remote", "add", "check-patch", repourl]) -subprocess.check_call(["git", "fetch", "check-patch", "master"], - stdout=subprocess.DEVNULL, - stderr=subprocess.DEVNULL) +subprocess.check_call(["git", "fetch", "check-patch", "master"]) ancestor = subprocess.check_output(["git", "merge-base", "check-patch/master", "HEAD"], |