You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@sling.apache.org by dk...@apache.org on 2023/11/13 15:13:50 UTC

(sling-org-apache-sling-committer-cli) 01/01: Adding more logging to failure to get CI status

This is an automated email from the ASF dual-hosted git repository.

dklco pushed a commit to branch SLING-12148
in repository https://gitbox.apache.org/repos/asf/sling-org-apache-sling-committer-cli.git

commit d35cad6b829e5f3c07675312ff2c967a88154cd8
Author: Dan Klco <kl...@adobe.com>
AuthorDate: Mon Nov 13 10:13:43 2023 -0500

    Adding more logging to failure to get CI status
---
 src/main/java/org/apache/sling/cli/impl/ci/CIStatusValidator.java | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/src/main/java/org/apache/sling/cli/impl/ci/CIStatusValidator.java b/src/main/java/org/apache/sling/cli/impl/ci/CIStatusValidator.java
index 25ca1c0..173f96b 100644
--- a/src/main/java/org/apache/sling/cli/impl/ci/CIStatusValidator.java
+++ b/src/main/java/org/apache/sling/cli/impl/ci/CIStatusValidator.java
@@ -139,8 +139,9 @@ public class CIStatusValidator {
         if (ciEndpoint == null) {
             return new ValidationResult(false, "Cannot extract a CI endpoint from " + artifactFilePath.getFileName());
         }
+        JsonObject status = null;
         try {
-            JsonObject status = fetchJson(ciEndpoint);
+            status = fetchJson(ciEndpoint);
 
             if ("pending".equals(status.get(PN_STATE).getAsString())
                     && status.get("statuses").getAsJsonArray().size() == 0) {
@@ -173,7 +174,8 @@ public class CIStatusValidator {
                 return new ValidationResult(false, message);
             }
         } catch (UnsupportedOperationException | IOException e) {
-            return new ValidationResult(false, "Failed to get CI Status: " + e.toString());
+            return new ValidationResult(false,
+                    "Failed to get CI Status: " + e.toString() + "\nUrl: " + ciEndpoint + "\nStatus Body: " + status);
         }
     }