Skip to content

Commit

Permalink
ci: enable an async test using cass_future_wait_timed
Browse files Browse the repository at this point in the history
This test needs to be run without valgrind, as it causes the client
request timeouts to appear.
  • Loading branch information
muzarski committed Aug 7, 2024
1 parent 25440dc commit 63a0d01
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 2 deletions.
5 changes: 4 additions & 1 deletion .github/workflows/build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -53,7 +53,10 @@ jobs:
:-PreparedTests.Integration_Cassandra_PreparedIDUnchangedDuringReprepare\
:ExecutionProfileTest.InvalidName"
run: valgrind --error-exitcode=123 --leak-check=full --errors-for-leak-kinds=definite ./cassandra-integration-tests --scylla --version=release:5.0.0 --category=CASSANDRA --verbose=ccm --gtest_filter="$Tests"

- name: Run timeout sensitive integration tests on Scylla 6.0.0 without valgrind
env:
Tests: "AsyncTests.Integration_Cassandra_Simple"
run: ./cassandra-integration-tests --scylla --version=release:6.0.0 --category=CASSANDRA --verbose=ccm --gtest_filter="$Tests"
- name: Upload test logs
uses: actions/upload-artifact@v3
if: success() || failure()
Expand Down
5 changes: 4 additions & 1 deletion .github/workflows/cassandra.yml
Original file line number Diff line number Diff line change
Expand Up @@ -57,7 +57,10 @@ jobs:
:SslTests.Integration_Cassandra_ReconnectAfterClusterCrashAndRestart\
:ExecutionProfileTest.InvalidName"
run: valgrind --error-exitcode=123 --leak-check=full --errors-for-leak-kinds=definite ./cassandra-integration-tests --version=4.0.7 --category=CASSANDRA --verbose=ccm --gtest_filter="$Tests"

- name: Run timeout sensitive integration tests on Cassandra 4.0.7 without valgrind
env:
Tests: "AsyncTests.Integration_Cassandra_Simple"
run: ./cassandra-integration-tests --version=4.0.7 --category=CASSANDRA --verbose=ccm --gtest_filter="$Tests"
- name: Upload test logs
uses: actions/upload-artifact@v3
if: success() || failure()
Expand Down

0 comments on commit 63a0d01

Please sign in to comment.