Fix tests

fsnotify · Oct 19, 2023 · 7483375 · 7483375
1 parent e6384f3
commit 7483375
Show file tree

Hide file tree

Showing 5 changed files with 224 additions and 215 deletions.
diff --git a/.circleci/config.yml b/.circleci/config.yml
@@ -22,10 +22,9 @@ jobs:
           name: test
           command: |
             uname -a
-            sysctl fs.inotify.max_user_watches fs.inotify.max_user_instances
-            ulimit -a
             go version
-            go test -parallel 1 -race ./...
+            FSNOTIFY_BUFFER=4096 go test -parallel 1 -race ./...
+                                 go test -parallel 1 -race ./...
 
   # iOS
   ios:
@@ -50,7 +49,8 @@ jobs:
             export PATH=$PATH:/usr/local/Cellar/go/*/bin
             uname -a
             go version
-            go test -parallel 1 -race ./...
+            FSNOTIFY_BUFFER=4096 go test -parallel 1 -race ./...
+                                 go test -parallel 1 -race ./...
 
   # This is just Linux x86_64; also need to get a Go with GOOS=android, but
   # there aren't any pre-built versions of that on the Go site. Idk, disable for
@@ -78,5 +78,6 @@ jobs:
   #           uname -a
   #           export PATH=/usr/local/go/bin:$PATH
   #           go version
-  #           go test -parallel 1 -race ./...
+  #           FSNOTIFY_BUFFER=4096 go test -parallel 1 -race ./...
+  #                                go test -parallel 1 -race ./...
   #
diff --git a/.cirrus.yml b/.cirrus.yml
@@ -9,4 +9,5 @@ freebsd_task:
       # run tests as user "cirrus" instead of root
     - pw useradd cirrus -m
     - chown -R cirrus:cirrus .
-    - sudo -u cirrus go test -parallel 1 -race ./...
+    - FSNOTIFY_BUFFER=4096 sudo --preserve-env=FSNOTIFY_BUFFER -u cirrus go test -parallel 1 -race ./...
+    -                      sudo --preserve-env=FSNOTIFY_BUFFER -u cirrus go test -parallel 1 -race ./...
diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml
@@ -6,71 +6,74 @@ on:
     branches: ['main', 'aix']
 
 jobs:
-  # Test Windows and Linux with the latest Go version and the oldest we support.
-  test:
+  linux:
     strategy:
       fail-fast: false
       matrix:
-        os:
-          - ubuntu-latest
-          - windows-latest
-        go:
-          - '1.17'
-          - '1.21'
+        os: ['ubuntu-latest']
+        go: ['1.17', '1.21']
     runs-on: ${{ matrix.os }}
     steps:
-      - name: checkout
-        uses: actions/checkout@v3
-
-      - name: setup Go
-        uses: actions/setup-go@v4
+      - uses: 'actions/checkout@v3'
+      - uses: 'actions/setup-go@v4'
         with:
           go-version: ${{ matrix.go }}
+      - name: test
+        run: |
+          FSNOTIFY_BUFFER=4096 go test -parallel 1 -race ./...
+                               go test -parallel 1 -race ./...
 
+  windows:
+    strategy:
+      fail-fast: false
+      matrix:
+        os: ['windows-latest']
+        go: ['1.17', '1.21']
+    runs-on: ${{ matrix.os }}
+    steps:
+      - uses: 'actions/checkout@v3'
+      - uses: 'actions/setup-go@v4'
+        with:
+          go-version: ${{ matrix.go }}
       - name: test
         run: |
           go test -parallel 1 -race ./...
+          set "FSNOTIFY_BUFFER=4096"
+          go test -parallel 1 -race ./...
 
   # Test gccgo
-  testgcc:
-    runs-on: ubuntu-22.04
-    name: test (ubuntu-22.04, gccgo 12.1)
+  gcc:
+    runs-on: 'ubuntu-22.04'
+    name:    'test (ubuntu-22.04, gccgo 12.1)'
     steps:
-      - name: checkout
-        uses: actions/checkout@v3
-
+      - uses: 'actions/checkout@v3'
       - name: test
         run: |
           sudo apt-get -y install gccgo-12
-          go-12 test -parallel 1 ./...
+          FSNOTIFY_BUFFER=4096 go-12 test -parallel 1 ./...
+                               go-12 test -parallel 1 ./...
 
   # Test only the latest Go version on macOS; we use the macOS builders for BSD
   # and illumos, and GitHub doesn't allow many of them to run concurrently. If
   # it works on Windows and Linux with Go 1.17, then it probably does on macOS
   # too.
-  testMacOS:
+  macos:
     name: test
     strategy:
       fail-fast: false
       matrix:
-        os:
-          - macos-11
-          - macos-13
-        go:
-          - '1.21'
+        os: ['macos-11', 'macos-13']
+        go: ['1.21']
     runs-on: ${{ matrix.os }}
     steps:
-      - name: checkout
-        uses: actions/checkout@v3
-
-      - name: setup Go
-        uses: actions/setup-go@v4
+      - uses: 'actions/checkout@v3'
+      - uses: 'actions/setup-go@v4'
         with:
           go-version: ${{ matrix.go }}
-
       - name: test
         run: |
-          go test -parallel 1 -race ./...
+          FSNOTIFY_BUFFER=4096 go test -parallel 1 -race ./...
+                               go test -parallel 1 -race ./...
 
   # OpenBSD; no -race as the VM doesn't include the comp set.
   #
@@ -79,57 +82,64 @@ jobs:
   #       so should probably look into that first. Go 1.19 is supposed to have a
   #       much faster race detector, so maybe waiting until we have that is
   #       enough.
-  testOpenBSD:
-    runs-on: macos-12
-    name: test (openbsd, 1.17)
+  openbsd:
+    runs-on: 'macos-12'
+    timeout-minutes: 30
+    name: 'test (openbsd, 1.17)'
     steps:
-      - uses: actions/checkout@v3
-      - name: test (openbsd, 1.17)
-        id: test
-        uses: vmactions/openbsd-vm@v0
+      - uses: 'actions/checkout@v3'
+      - name: 'test (openbsd, 1.17)'
+        id:   'openbsd'
+        uses: 'vmactions/openbsd-vm@v0'
         with:
           prepare: pkg_add go
           run: |
             useradd -mG wheel action
-            su action -c 'go test -parallel 1 ./...'
+            FSNOTIFY_BUFFER=4096 su action -c 'go test -parallel 1 ./...'
+                                 su action -c 'go test -parallel 1 ./...'
 
   # NetBSD
-  testNetBSD:
+  netbsd:
     runs-on: macos-12
+    timeout-minutes: 30
     name: test (netbsd, 1.20)
     steps:
-      - uses: actions/checkout@v3
-      - name: test (netbsd, 1.20)
-        id: test
-        uses: vmactions/netbsd-vm@v0
+      - uses: 'actions/checkout@v3'
+      - name: 'test (netbsd, 1.20)'
+        id:   'netbsd'
+        uses: 'vmactions/netbsd-vm@v0'
         with:
           prepare: pkg_add go
           # TODO: no -race for the same reason as OpenBSD (the timing; it does run).
           run: |
             useradd -mG wheel action
-            su action -c 'go120 test -parallel 1 ./...'
+            FSNOTIFY_BUFFER=4096 su action -c 'go120 test -parallel 1 ./...'
+                                 su action -c 'go120 test -parallel 1 ./...'
 
   # illumos
-  testillumos:
+  illumos:
     runs-on: macos-12
+    timeout-minutes: 30
     name: test (illumos, 1.19)
     steps:
-    - uses: actions/checkout@v3
-    - name: test (illumos, 1.19)
-      id: test
-      uses: papertigers/illumos-vm@r38
+    - uses: 'actions/checkout@v3'
+    - name: 'test (illumos, 1.19)'
+      id:   'illumos'
+      uses: 'papertigers/illumos-vm@r38'
       with:
         prepare: |
           pkg install go-119
         run: |
           useradd action
           export GOCACHE=/tmp/go-cache
           export GOPATH=/tmp/go-path
-          su action -c '/opt/ooce/go-1.19/bin/go test -parallel 1 ./...'
+          FSNOTIFY_BUFFER=4096 su action -c '/opt/ooce/go-1.19/bin/go test -parallel 1 ./...'
+                               su action -c '/opt/ooce/go-1.19/bin/go test -parallel 1 ./...'
 
   # Older Debian 6, for old Linux kernels.
-  testDebian6:
+  debian6:
     runs-on: macos-12
+    timeout-minutes: 30
     name: test (debian6, 1.19)
     strategy:
       fail-fast: false
@@ -149,20 +159,18 @@ jobs:
         with:
           go-version: '1.19'
 
-      - name: test (debian6, 1.19)
-        id: test
+      - name: 'test (debian6, 1.19)'
+        id:   'debian6'
         run: |
           cp -f .github/workflows/Vagrantfile.debian6 Vagrantfile
           export GOOS=linux
           export GOARCH=amd64
           for p in $(go list ./...); do
-            go test -c -o ${p//\//-}.test $p
+            FSNOTIFY_BUFFER=4096 go test -c -o ${p//\//-}.test $p
+                                 go test -c -o ${p//\//-}.test $p
           done
           vagrant up
-
-          vagrant ssh -c 'uname -a'
-          vagrant ssh -c 'tail /proc/sys/fs/inotify/max_user_watches /proc/sys/fs/inotify/max_user_instances'
-          vagrant ssh -c 'ulimit -a'
           for t in *.test; do
-            vagrant ssh -c "/vagrant/$t -test.parallel 1"
+            FSNOTIFY_BUFFER=4096 vagrant ssh -c "/vagrant/$t -test.parallel 1"
+                                 vagrant ssh -c "/vagrant/$t -test.parallel 1"
           done