From da5884ebe31b7b7023ffbaf552661cd6819af4b3 Mon Sep 17 00:00:00 2001 From: Peter Bengtsson Date: Mon, 13 Jun 2022 14:51:09 -0400 Subject: [PATCH] reduce linkinator concurrency from 300 to 100 (#28176) * reduce linkinator concurrency from 300 to 100 * reduce it and use self-hosted * try again * actually do concurrency of 10 * move the override to the Action workflow * typo * avoid memory bloat * rename * back to 100 * set LINKINATOR_CONCURRENCY on the right thing * remove some debugging --- .github/workflows/check-all-english-links.yml | 12 ++++-------- 1 file changed, 4 insertions(+), 8 deletions(-) diff --git a/.github/workflows/check-all-english-links.yml b/.github/workflows/check-all-english-links.yml index 82909656fb..baae8c5bd6 100644 --- a/.github/workflows/check-all-english-links.yml +++ b/.github/workflows/check-all-english-links.yml @@ -58,9 +58,6 @@ jobs: REQUEST_TIMEOUT: 20000 # Don't care about CDN caching image URLs DISABLE_REWRITE_ASSET_URLS: true - # The default is 300 which works OK on a fast macbook pro - # but so well in Actions. - LINKINATOR_CONCURRENCY: 100 run: | node server.mjs > /tmp/stdout.log 2> /tmp/stderr.log & sleep 5 @@ -75,6 +72,10 @@ jobs: cat /tmp/stderr.log - name: Run script + env: + # The default is 300 which works OK on a fast macbook pro + # but not so well in Actions. + LINKINATOR_CONCURRENCY: 100 run: | script/check-english-links.js > broken_links.md @@ -86,11 +87,6 @@ jobs: # # https://docs.github.com/actions/reference/context-and-expression-syntax-for-github-actions#job-status-check-functions - - if: ${{ failure() }} - name: Debug broken_links.md - run: | - ls -lh broken_links.md - wc -l broken_links.md - uses: actions/upload-artifact@6673cd052c4cd6fcf4b4e6e60ea986c889389535 if: ${{ failure() }} with: