Skip to content

Commit cd3dee4

Browse files
committed
tests: ynl: use tmp dir for transient outputs
Use temp dir for files we only need temporarily. This saves us upload and delete cycles if results are a remote filesystem. Signed-off-by: Jakub Kicinski <[email protected]>
1 parent 8d32947 commit cd3dee4

File tree

1 file changed

+10
-9
lines changed

1 file changed

+10
-9
lines changed

tests/series/ynl/ynl.sh

Lines changed: 10 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -40,33 +40,34 @@ fi
4040
##################################################################
4141
echo " ====== 3/ Generate diffs for user codegen ======"
4242

43-
mkdir $RESULTS_DIR/old-code
43+
TEMP_DIR=$(mktemp -d /tmp/ynl_build-tmp.XXXXXX)
44+
45+
mkdir $TEMP_DIR/old-code
4446
git checkout -q $BRANCH_BASE
4547
make -C tools/net/ynl/generated/ distclean
4648
make -C tools/net/ynl/generated/ -j $ncpu
47-
cp tools/net/ynl/generated/*.[ch] $RESULTS_DIR/old-code/
49+
cp tools/net/ynl/generated/*.[ch] $TEMP_DIR/old-code/
4850

49-
mkdir $RESULTS_DIR/new-code
51+
mkdir $TEMP_DIR/new-code
5052
git checkout -q $HEAD
5153
make -C tools/net/ynl/generated/ distclean
5254
make -C tools/net/ynl/generated/ -j $ncpu
53-
cp tools/net/ynl/generated/*.[ch] $RESULTS_DIR/new-code/
55+
cp tools/net/ynl/generated/*.[ch] $TEMP_DIR/new-code/
5456

5557
git diff --no-index --stat \
56-
$RESULTS_DIR/old-code/ $RESULTS_DIR/new-code/ > $RESULTS_DIR/diff-stat
58+
$TEMP_DIR/old-code/ $TEMP_DIR/new-code/ > $RESULTS_DIR/diff-stat
5759
git diff --no-index \
58-
$RESULTS_DIR/old-code/ $RESULTS_DIR/new-code/ > $RESULTS_DIR/diff
60+
$TEMP_DIR/old-code/ $TEMP_DIR/new-code/ > $RESULTS_DIR/diff
5961

6062
git diff --no-index --exit-code \
61-
$RESULTS_DIR/old-code/ $RESULTS_DIR/new-code/ >> /dev/null
63+
$TEMP_DIR/old-code/ $TEMP_DIR/new-code/ >> /dev/null
6264
if [ $? -eq 0 ]; then
6365
echo "no diff in generated;" >&$DESC_FD
6466
else
6567
echo "GEN HAS DIFF $(cat $RESULTS_DIR/diff-stat | tail -1);" >&$DESC_FD
6668
fi
6769

68-
rm -rf $RESULTS_DIR/old-code/ $RESULTS_DIR/new-code/
69-
70+
rm -rf $TEMP_DIR
7071
rm $tmpfile
7172

7273
exit $rc

0 commit comments

Comments
 (0)