From 8ee29f3edd183c7f99dc594dbbf20ef47b0762de Mon Sep 17 00:00:00 2001 From: Elliot Winkler Date: Sun, 4 Feb 2024 22:00:20 -0700 Subject: [PATCH] Revert "Use Zeus to run integration tests on CI (#212)" This reverts commit 9e461603c205a6de7d2fa1ceb40db8022a56947d. Unfortunately, Zeus is just not reliable enough to run on CI. It's not even reliable locally. Sometimes tests will hang for no reason, and sometimes Zeus will fail with errors like `read unix ->: recvmsg: message too long`. And unfortunately, once Zeus fails, the best way to fix it is to restart it. This isn't possible to do in CI tests, and adding some kind of "with Zeus, then without Zeus" strategy to tests would add complexity I don't want. I'll have to find a better way. --- .github/workflows/super_diff.yml | 11 ----------- 1 file changed, 11 deletions(-) diff --git a/.github/workflows/super_diff.yml b/.github/workflows/super_diff.yml index 25048ffe..73d7a9d4 100644 --- a/.github/workflows/super_diff.yml +++ b/.github/workflows/super_diff.yml @@ -103,16 +103,5 @@ jobs: with: ruby-version: ${{ matrix.ruby }} bundler-cache: true - - name: Install Zeus - run: gem install zeus - - name: Start Zeus - uses: JarvusInnovations/background-action@v1 - with: - run: zeus start - wait-on: | - socket:.zeus.sock - file:.zeus.sock - wait-for: 15s - log-output-if: failure - name: Run tests run: bundle exec rake --trace