Merge pull request #1128 from redis/lower-ci-timeout

Lower client timeout on CI

Author: byroot

.github/workflows/test.yaml

@@ -10,7 +10,7 @@ on:
 jobs:
   lint:
     name: Rubocop
-    timeout-minutes: 30
+    timeout-minutes: 15
     runs-on: ubuntu-latest
     steps:
       - name: Check out code
@@ -29,19 +29,15 @@ jobs:
 
   rubies:
     name: Ruby
-    timeout-minutes: 30
+    timeout-minutes: 15
     strategy:
       fail-fast: false
       matrix:
-        ruby: ["3.1", "3.0", "2.7", "2.6", "2.5", "jruby-9.3.6.0", "truffleruby-22.2.0"]
+        ruby: ["3.1", "3.0", "2.7", "2.6", "2.5", "jruby-9.3.6.0"]
     runs-on: ubuntu-latest
     env:
-      VERBOSE: "true"
-      TIMEOUT: "30"
       LOW_TIMEOUT: "0.01"
-      DRIVER: ruby
       REDIS_BRANCH: "7.0"
-      TRUFFLERUBYOPT: "--engine.Mode=latency"
     steps:
       - name: Check out code
         uses: actions/checkout@v3
@@ -68,17 +64,53 @@ jobs:
       - name: Shutting down Redis
         run: make stop
 
+  truffle:
+    name: TruffleRuby
+    timeout-minutes: 15
+    strategy:
+      fail-fast: false
+      matrix:
+        suite: ["redis", "distributed"]
+    runs-on: ubuntu-latest
+    env:
+      LOW_TIMEOUT: "0.01"
+      REDIS_BRANCH: "7.0"
+      TRUFFLERUBYOPT: "--engine.Mode=latency"
+    steps:
+      - name: Check out code
+        uses: actions/checkout@v3
+      - name: Print environment variables
+        run: |
+          echo "TIMEOUT=${TIMEOUT}"
+          echo "LOW_TIMEOUT=${LOW_TIMEOUT}"
+          echo "DRIVER=${DRIVER}"
+          echo "REDIS_BRANCH=${REDIS_BRANCH}"
+      - name: Set up Ruby
+        uses: ruby/setup-ruby@v1
+        with:
+          ruby-version: truffleruby-22.2.0
+          bundler-cache: true
+      - name: Cache local temporary directory
+        uses: actions/cache@v3
+        with:
+          path: tmp
+          key: "local-tmp-redis-7.0-on-ubuntu-latest"
+      - name: Booting up Redis
+        run: make start
+      - name: Test
+        run: bundle exec rake test:${{ matrix.suite }}
+      - name: Shutting down Redis
+        run: make stop
+
   drivers:
     name: Driver
-    timeout-minutes: 30
+    timeout-minutes: 15
     strategy:
       fail-fast: false
       matrix:
         driver: ["hiredis"]
     runs-on: ubuntu-latest
     env:
-      VERBOSE: "true"
-      TIMEOUT: "30"
       LOW_TIMEOUT: "0.01"
       DRIVER: ${{ matrix.driver }}
       REDIS_BRANCH: "7.0"
@@ -110,17 +142,14 @@ jobs:
 
   redises:
     name: Redis
-    timeout-minutes: 30
+    timeout-minutes: 15
     strategy:
       fail-fast: false
       matrix:
         redis: ["6.2", "6.0", "5.0"]
     runs-on: ubuntu-latest
     env:
-      VERBOSE: "true"
-      TIMEOUT: "30"
       LOW_TIMEOUT: "0.14"
-      DRIVER: ruby
       REDIS_BRANCH: ${{ matrix.redis }}
     steps:
       - name: Check out code
@@ -150,15 +179,12 @@ jobs:
 
   sentinel:
     name: Sentinel
-    timeout-minutes: 30
+    timeout-minutes: 15
     strategy:
       fail-fast: false
     runs-on: ubuntu-latest
     env:
-      VERBOSE: "true"
-      TIMEOUT: "30"
       LOW_TIMEOUT: "0.14"
-      DRIVER: ruby
       REDIS_BRANCH: "7.0"
     steps:
       - name: Check out code
@@ -188,15 +214,13 @@ jobs:
 
   cluster:
     name: Cluster
-    timeout-minutes: 30
+    timeout-minutes: 15
     strategy:
       fail-fast: false
     runs-on: ubuntu-latest
     env:
-      VERBOSE: "true"
-      TIMEOUT: "30"
+      TIMEOUT: "15"
       LOW_TIMEOUT: "0.14"
-      DRIVER: ruby
       REDIS_BRANCH: "7.0"
       BUNDLE_GEMFILE: redis_cluster/Gemfile
     steps:

test/redis/internals_test.rb

@@ -9,6 +9,10 @@ def test_large_payload
     # see: https://github.com/redis/redis-rb/issues/962
     # large payloads will trigger write_nonblock to write a portion
     # of the payload in connection/ruby.rb _write_to_socket
+
+    # We use a larger timeout for TruffleRuby
+    # https://github.com/redis/redis-rb/pull/1128#issuecomment-1218490684
+    r = init(_new_client(timeout: TIMEOUT * 5))
     large = "\u3042" * 4_000_000
     r.setex("foo", 10, large)
     result = r.get("foo")