Skip to content

Commit

Permalink
ZTS: Fix summary page creation again - second try
Browse files Browse the repository at this point in the history
In PR openzfs#16599 I used 'return' like in C - which is wrong :/
This fix generates the summary as needed.

Reviewed-by: Brian Behlendorf <behlendorf1@llnl.gov>
Signed-off-by: Tino Reichardt <milky-zfs@mcmilk.de>
Closes openzfs#16611
  • Loading branch information
mcmilk authored and ixhamza committed Oct 8, 2024
1 parent 3398168 commit 29fcee5
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions .github/workflows/scripts/qemu-9-summary-page.sh
Original file line number Diff line number Diff line change
Expand Up @@ -11,12 +11,10 @@ function output() {
}

function outfile() {
test -s "$1" || return
cat "$1" >> "out-$logfile.md"
}

function outfile_plain() {
test -s "$1" || return
output "<pre>"
cat "$1" >> "out-$logfile.md"
output "</pre>"
Expand Down Expand Up @@ -45,6 +43,8 @@ if [ ! -f out-1.md ]; then
tar xf "$tarfile"
test -s env.txt || continue
source env.txt
# when uname.txt is there, the other files are also ok
test -s uname.txt || continue
output "\n## Functional Tests: $OSNAME\n"
outfile_plain uname.txt
outfile_plain summary.txt
Expand Down

0 comments on commit 29fcee5

Please sign in to comment.