[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PULL 07/14] scripts/ci/gitlab-pipeline-status: give more information on
From: |
Thomas Huth |
Subject: |
[PULL 07/14] scripts/ci/gitlab-pipeline-status: give more information on failures |
Date: |
Tue, 9 Mar 2021 11:46:10 +0100 |
From: Cleber Rosa <crosa@redhat.com>
When an HTTP GET request fails, it's useful to go beyond the "not
successful" message, and show the code returned by the server.
Signed-off-by: Cleber Rosa <crosa@redhat.com>
Reviewed-by: Alex Bennée <alex.bennee@linaro.org>
Reviewed-by: Wainer dos Santos Moschetta <wainersm@redhat.com>
Message-Id: <20210222193240.921250-3-crosa@redhat.com>
Signed-off-by: Thomas Huth <thuth@redhat.com>
---
scripts/ci/gitlab-pipeline-status | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/scripts/ci/gitlab-pipeline-status
b/scripts/ci/gitlab-pipeline-status
index 0c1e8bd8a7..ad62ab3cfc 100755
--- a/scripts/ci/gitlab-pipeline-status
+++ b/scripts/ci/gitlab-pipeline-status
@@ -56,7 +56,9 @@ def get_json_http_response(url):
connection.request('GET', url=url)
response = connection.getresponse()
if response.code != http.HTTPStatus.OK:
- raise CommunicationFailure("Failed to receive a successful response")
+ msg = "Received unsuccessful response: %s (%s)" % (response.code,
+ response.reason)
+ raise CommunicationFailure(msg)
return json.loads(response.read())
--
2.27.0
- [PULL 00/14] Gitlab, testing and misc patches, Thomas Huth, 2021/03/09
- [PULL 02/14] docker: Alpine build job depends on Alpine container, Thomas Huth, 2021/03/09
- [PULL 03/14] docker: EDK2 build job depends on EDK2 container, Thomas Huth, 2021/03/09
- [PULL 01/14] qtest: delete superfluous inclusions of qtest.h, Thomas Huth, 2021/03/09
- [PULL 04/14] docker: OpenSBI build job depends on OpenSBI container, Thomas Huth, 2021/03/09
- [PULL 05/14] meson: Re-enable the possibility to run "make check SPEED=slow", Thomas Huth, 2021/03/09
- [PULL 06/14] scripts/ci/gitlab-pipeline-status: split utlity function for HTTP GET, Thomas Huth, 2021/03/09
- [PULL 07/14] scripts/ci/gitlab-pipeline-status: give more information on failures,
Thomas Huth <=
- [PULL 08/14] scripts/ci/gitlab-pipeline-status: give more info when pipeline not found, Thomas Huth, 2021/03/09
- [PULL 09/14] libqos/qgraph: format qgraph comments for sphinx documentation, Thomas Huth, 2021/03/09
- [PULL 10/14] docs/devel/qgraph: improve qgraph documentation, Thomas Huth, 2021/03/09
- [PULL 11/14] Acceptance Tests: restore downloading of VM images, Thomas Huth, 2021/03/09
- [PULL 12/14] Acceptance Tests: restore filtering of tests by target arch, Thomas Huth, 2021/03/09
- [PULL 14/14] bsd-user: Add new maintainers, Thomas Huth, 2021/03/09
- [PULL 13/14] Remove deprecated target tilegx, Thomas Huth, 2021/03/09
- Re: [PULL 00/14] Gitlab, testing and misc patches, no-reply, 2021/03/09
- Re: [PULL 00/14] Gitlab, testing and misc patches, Peter Maydell, 2021/03/10