Skip to content

Commit

Permalink
disable frontend + merge sqlsmith-test workflow back into main-cron
Browse files Browse the repository at this point in the history
  • Loading branch information
kwannoel committed Sep 29, 2023
1 parent b268150 commit 28e26e5
Show file tree
Hide file tree
Showing 4 changed files with 31 additions and 58 deletions.
4 changes: 3 additions & 1 deletion ci/scripts/cron-fuzz-test.sh
Original file line number Diff line number Diff line change
Expand Up @@ -4,12 +4,14 @@
set -euo pipefail

source ci/scripts/common.sh
export RUN_SQLSMITH=0

# NOTE(kwannoel): Disabled because there's some breakage after #12485,
# see https://github.com/risingwavelabs/risingwave/issues/12577.
# Frontend is relatively stable, e2e fuzz test will cover the same cases also,
# so we can just disable it.
# export RUN_SQLSMITH_FRONTEND=1

export RUN_SQLSMITH=1
export SQLSMITH_COUNT=1000
export TEST_NUM=100
source ci/scripts/run-fuzz-test.sh
27 changes: 13 additions & 14 deletions ci/scripts/pr-fuzz-test.sh
Original file line number Diff line number Diff line change
Expand Up @@ -16,28 +16,27 @@ NOT_MATCHES="\.md"
CHANGED=$(git diff --name-only origin/main | grep -v "$NOT_MATCHES" | grep "$MATCHES")
set -e

# Always run sqlsmith frontend tests
# NOTE(kwannoel): Disabled because there's some breakage after #12485,
# see https://github.com/risingwavelabs/risingwave/issues/12577.
# Frontend is relatively stable, e2e fuzz test will cover the same cases also,
# so we can just disable it.
# export RUN_SQLSMITH_FRONTEND=1

export RUN_SQLSMITH=1
export SQLSMITH_COUNT=100

# Run e2e tests if changes to sqlsmith source files detected.
# NOTE(kwannoel): Keep this here in-case we ever want to revert.
#if [[ -n "$CHANGED" ]]; then
# echo "--- Checking whether to run all sqlsmith tests"
# echo "origin/main SHA: $(git rev-parse origin/main)"
# echo "Changes to Sqlsmith source files detected:"
# echo "$CHANGED"
# export RUN_SQLSMITH=1
# export SQLSMITH_COUNT=100
# export TEST_NUM=32
# echo "Enabled Sqlsmith tests."
#else
# export RUN_SQLSMITH=0
#fi
if [[ -n "$CHANGED" ]]; then
echo "--- Checking whether to run all sqlsmith tests"
echo "origin/main SHA: $(git rev-parse origin/main)"
echo "Changes to Sqlsmith source files detected:"
echo "$CHANGED"
export RUN_SQLSMITH=1
export SQLSMITH_COUNT=100
export TEST_NUM=32
echo "Enabled Sqlsmith tests."
else
export RUN_SQLSMITH=0
fi

source ci/scripts/run-fuzz-test.sh
15 changes: 15 additions & 0 deletions ci/workflows/pull-request.yml
Original file line number Diff line number Diff line change
Expand Up @@ -491,3 +491,18 @@ steps:
- ./ci/plugins/upload-failure-logs
timeout_in_minutes: 25
retry: *auto-retry

- label: "fuzz test"
command: "ci/scripts/pr-fuzz-test.sh -p ci-dev"
depends_on:
- "build"
- "build-simulation"
plugins:
- ./ci/plugins/swapfile
- docker-compose#v4.9.0:
run: rw-build-env
config: ci/docker-compose.yml
mount-buildkite-agent: true
- ./ci/plugins/upload-failure-logs
timeout_in_minutes: 15
retry: *auto-retry
43 changes: 0 additions & 43 deletions ci/workflows/sqlsmith-tests.yml

This file was deleted.

0 comments on commit 28e26e5

Please sign in to comment.