[PATCH v2 0/2] Minimise CI test log

[PATCH v2 0/2] Minimise CI test log

[Anton Mikanovich]

There are some redundant output parts in Avocado logs we can remove.

Anton Mikanovich (2):
  CI: Trace test only
  CI: Ignore POLLHUP events in log

 scripts/ci_build.sh               | 6 +++---
 testsuite/build_test/cibuilder.py | 2 ++
 2 files changed, 5 insertions(+), 3 deletions(-)

-- 
2.25.1

[PATCH v2 1/2] CI: Trace test only

[Anton Mikanovich]

No minimise test log hide traces of prepare operations.

Signed-off-by: Anton Mikanovich <amikan@ilbers.de>
---
 scripts/ci_build.sh | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/scripts/ci_build.sh b/scripts/ci_build.sh
index 5ff455e..e9ba034 100755
--- a/scripts/ci_build.sh
+++ b/scripts/ci_build.sh
@@ -113,9 +113,6 @@ if [ -z "$REPRO_BUILD" ]; then
     TAGS="$TAGS,-repro"
 fi
 
-# the real stuff starts here, trace commands from now on
-set -x
-
 # Provide working path
 mkdir -p .config/avocado
 cat <<EOF > .config/avocado/avocado.conf
@@ -127,6 +124,9 @@ logs_dir = $(realpath $BUILD_DIR)/job-results
 EOF
 export VIRTUAL_ENV="./"
 
+# the real stuff starts here, trace commands from now on
+set -x
+
 avocado $VERBOSE run "$BUILD_TEST_DIR/build_test.py" \
     -t $TAGS --test-runner=runner --disable-sysinfo \
     -p build_dir="$BUILD_DIR" -p quiet=$QUIET -p cross=$CROSS_BUILD
-- 
2.25.1

[PATCH v2 2/2] CI: Ignore POLLHUP events in log

[Anton Mikanovich]

Print only POLLIN event buffers to not show empty lines at POLLHUP
events at the end of execution.
This will reduce test log size.

Signed-off-by: Anton Mikanovich <amikan@ilbers.de>
---
 testsuite/build_test/cibuilder.py | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/testsuite/build_test/cibuilder.py b/testsuite/build_test/cibuilder.py
index baa7185..94786c7 100644
--- a/testsuite/build_test/cibuilder.py
+++ b/testsuite/build_test/cibuilder.py
@@ -92,6 +92,8 @@ class CIBuilder(Test):
             while p1.poll() is None:
                 events = poller.poll(1000)
                 for fd, event in events:
+                    if event != select.POLLIN:
+                        continue
                     if fd == p1.stdout.fileno():
                         self.log.info(p1.stdout.readline().rstrip())
                     if fd == p1.stderr.fileno():
-- 
2.25.1

Re: [PATCH v2 0/2] Minimise CI test log

[Anton Mikanovich]

04.11.2021 08:58, Anton Mikanovich wrote:
> There are some redundant output parts in Avocado logs we can remove.
>
> Anton Mikanovich (2):
>    CI: Trace test only
>    CI: Ignore POLLHUP events in log
>
>   scripts/ci_build.sh               | 6 +++---
>   testsuite/build_test/cibuilder.py | 2 ++
>   2 files changed, 5 insertions(+), 3 deletions(-)
>
Applied to next.

-- 
Anton Mikanovich
Promwad Ltd.
External service provider of ilbers GmbH
Maria-Merian-Str. 8
85521 Ottobrunn, Germany
+49 (89) 122 67 24-0
Commercial register Munich, HRB 214197
General Manager: Baurzhan Ismagulov