Skip to content

Commit

Permalink
ci: Remove SauceLabs retries (#3660)
Browse files Browse the repository at this point in the history
SauceLabs retries aren't necessary anymore because SauceLabs isn't
failing that often anymore. The retries queue many jobs in SauceLabs.
Instead, we should take failing SauceLabs tests seriously and try to fix
them or disable a flaky test if necessary.
  • Loading branch information
philipphofmann authored Feb 19, 2024
1 parent 9e8c574 commit 4c2dc5f
Showing 1 changed file with 1 addition and 2 deletions.
3 changes: 1 addition & 2 deletions .github/workflows/saucelabs-UI-tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -114,9 +114,8 @@ jobs:

- run: npm install -g [email protected]

# As Sauce Labs is a bit flaky we retry 5 times
- name: Run Tests in SauceLab
env:
SAUCE_USERNAME: ${{ secrets.SAUCE_USERNAME }}
SAUCE_ACCESS_KEY: ${{ secrets.SAUCE_ACCESS_KEY }}
run: for i in {1..5}; do saucectl run --select-suite ${{ matrix.suite }} && break ; done
run: saucectl run --select-suite ${{ matrix.suite }}

0 comments on commit 4c2dc5f

Please sign in to comment.