Skip to content

Commit

Permalink
[FLINK-23965][e2e] Store logs under FLINK_DIR/log
Browse files Browse the repository at this point in the history
This commit reverts the change that we store the logs under FLINK_DIR/logs because this can
make the e2e tests fail if the directory does not exist. Now we store the logs under FLINK_DIR/log

This closes apache#16981.
  • Loading branch information
tillrohrmann committed Aug 26, 2021
1 parent 8bf1fbd commit e5de4a2
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion flink-end-to-end-tests/run-nightly-tests.sh
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,7 @@ if [ -z "$FLINK_DIR" ] ; then
fi

if [ -z "$FLINK_LOG_DIR" ] ; then
export FLINK_LOG_DIR="$FLINK_DIR/logs"
export FLINK_LOG_DIR="$FLINK_DIR/log"
fi

# On Azure CI, use artifacts dir
Expand Down
2 changes: 1 addition & 1 deletion flink-end-to-end-tests/run-single-test.sh
Original file line number Diff line number Diff line change
Expand Up @@ -47,7 +47,7 @@ if [ -z "$FLINK_DIR" ] ; then
fi

if [ -z "$FLINK_LOG_DIR" ] ; then
export FLINK_LOG_DIR="$FLINK_DIR/logs"
export FLINK_LOG_DIR="$FLINK_DIR/log"
fi

source "${END_TO_END_DIR}/../tools/ci/maven-utils.sh"
Expand Down
2 changes: 1 addition & 1 deletion flink-end-to-end-tests/test-scripts/common.sh
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,7 @@ if [[ -z $FLINK_DIR ]]; then
fi

if [ -z "$FLINK_LOG_DIR" ] ; then
export FLINK_LOG_DIR="$FLINK_DIR/logs"
export FLINK_LOG_DIR="$FLINK_DIR/log"
fi

case "$(uname -s)" in
Expand Down

0 comments on commit e5de4a2

Please sign in to comment.