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

sergehuber pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/unomi.git


The following commit(s) were added to refs/heads/master by this push:
     new f95d12358 UNOMI-937: Diagnostic fixes that missed the merge window 
(#765)
f95d12358 is described below

commit f95d123586e897f60f3c0abd7cff69d380ee8123
Author: Serge Huber <[email protected]>
AuthorDate: Thu May 28 14:28:48 2026 +0200

    UNOMI-937: Diagnostic fixes that missed the merge window (#765)
---
 build.sh                                                           | 7 +++++--
 .../test/java/org/apache/unomi/itests/graphql/GraphQLListIT.java   | 5 +++++
 setenv.sh                                                          | 7 +++++--
 3 files changed, 15 insertions(+), 4 deletions(-)

diff --git a/build.sh b/build.sh
index e6f1ae619..4b8251326 100755
--- a/build.sh
+++ b/build.sh
@@ -19,13 +19,15 @@
 
################################################################################
 
 set -e  # Exit on error
-# Keep trap arguments small: passing full $BASH_COMMAND can exceed ARG_MAX 
after a failed mvn invocation.
-trap 'handle_error $? $LINENO' ERR
+# Truncate $BASH_COMMAND in the trap (where it correctly reflects the failing
+# command) to avoid ARG_MAX limits after a failed mvn invocation.
+trap 'handle_error $? $LINENO "${BASH_COMMAND:0:200}"' ERR
 
 # Error handling function
 handle_error() {
     local exit_code=$1
     local line_no=$2
+    local failed_cmd=$3
 
     cat << "EOF"
      _____ ____  ____   ___  ____
@@ -36,6 +38,7 @@ handle_error() {
 
 EOF
     echo "Error occurred in:"
+    echo "  Command: $failed_cmd"
     echo "  Line: $line_no"
     echo "  Exit code: $exit_code"
     exit $exit_code
diff --git 
a/itests/src/test/java/org/apache/unomi/itests/graphql/GraphQLListIT.java 
b/itests/src/test/java/org/apache/unomi/itests/graphql/GraphQLListIT.java
index 139c19063..60a9a8507 100644
--- a/itests/src/test/java/org/apache/unomi/itests/graphql/GraphQLListIT.java
+++ b/itests/src/test/java/org/apache/unomi/itests/graphql/GraphQLListIT.java
@@ -21,10 +21,14 @@ import org.apache.unomi.api.Profile;
 import org.apache.unomi.lists.UserList;
 import org.junit.Assert;
 import org.junit.Test;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
 import java.util.Objects;
 
 public class GraphQLListIT extends BaseGraphQLIT {
 
+    private static final Logger LOGGER = 
LoggerFactory.getLogger(GraphQLListIT.class);
+
     @Test
     public void testCRUD() throws Exception {
         Profile persistedProfile = null;
@@ -82,6 +86,7 @@ public class GraphQLListIT extends BaseGraphQLIT {
                         try (CloseableHttpResponse response = 
post("graphql/list/find-lists.json")) {
                             return ResponseContext.parse(response.getEntity());
                         } catch (Exception e) {
+                            LOGGER.warn("find-lists poll attempt failed: {}", 
e.getMessage());
                             return null;
                         }
                     },
diff --git a/setenv.sh b/setenv.sh
index aa780461e..115a5f848 100755
--- a/setenv.sh
+++ b/setenv.sh
@@ -17,12 +17,15 @@
 #    limitations under the License.
 #
 
################################################################################
-# Quiet evaluate: avoid capturing Maven download lines into the environment 
(breaks CI with ARG_MAX).
-export UNOMI_VERSION="$(mvn -B -q -DforceStdout help:evaluate 
-Dexpression=project.version -DinteractiveMode=false 2>/dev/null)"
+_mvn_err=$(mktemp)
+export UNOMI_VERSION="$(mvn -B -q -DforceStdout help:evaluate 
-Dexpression=project.version -DinteractiveMode=false 2>"$_mvn_err")"
 if [ -z "$UNOMI_VERSION" ]; then
     echo "Failed to detect project version from Maven" >&2
+    cat "$_mvn_err" >&2
+    rm -f "$_mvn_err"
     exit 1
 fi
+rm -f "$_mvn_err"
 echo "Detected project version=$UNOMI_VERSION"
 export KARAF_VERSION=4.4.8
 # Uncomment the following line if you need Apache Unomi to start automatically 
at the first start

Reply via email to