[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 0/3] gitlab-pipeline-status script: provide more information
From: |
Wainer dos Santos Moschetta |
Subject: |
Re: [PATCH 0/3] gitlab-pipeline-status script: provide more information on errors |
Date: |
Tue, 23 Feb 2021 11:52:17 -0300 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.7.0 |
Hi Cleber,
In case you need to send a v2, mind to add the following patch together?
commit 3c4ed8a78e096e4d7df0398c29887a9d468ae120 (HEAD -> gitlab_runners)
Author: Wainer dos Santos Moschetta <wainersm@redhat.com>
Date: Tue Feb 23 11:26:08 2021 -0300
scripts/ci/gitlab-pipeline-status: Handle ValueError exceptions nicely
With this change, when getting the local branch, it will handle nicely
any threw ValueError exception instead of print the stack trace.
Signed-off-by: Wainer dos Santos Moschetta <wainersm@redhat.com>
diff --git a/scripts/ci/gitlab-pipeline-status
b/scripts/ci/gitlab-pipeline-status
index 924db327ff..6177df973a 100755
--- a/scripts/ci/gitlab-pipeline-status
+++ b/scripts/ci/gitlab-pipeline-status
@@ -160,7 +160,11 @@ def main():
args = parser.parse_args()
if not args.commit:
- args.commit = get_local_branch_commit(args.branch)
+ try:
+ args.commit = get_local_branch_commit(args.branch)
+ except ValueError as error:
+ print("ERROR: %s" % error)
+ sys.exit(1)
success = False
try:
On 2/22/21 4:32 PM, Cleber Rosa wrote:
When things go wrong with the GitLab API requests, it's useful to give
users more information about the possible causes.
Cleber Rosa (3):
scripts/ci/gitlab-pipeline-status: split utlity function for HTTP GET
scripts/ci/gitlab-pipeline-status: give more information on failures
scripts/ci/gitlab-pipeline-status: give more info when pipeline not
found
scripts/ci/gitlab-pipeline-status | 25 ++++++++++++++++++-------
1 file changed, 18 insertions(+), 7 deletions(-)