Skip to content

Commit f74ad0b

Browse files
authored
chore: move ulimit increase to per-step execution in CI workflows (#3004)
chore: increase ulimit per shell
1 parent 55c1caf commit f74ad0b

File tree

1 file changed

+11
-27
lines changed

1 file changed

+11
-27
lines changed

.github/workflows/health_checks.yml

Lines changed: 11 additions & 27 deletions
Original file line numberDiff line numberDiff line change
@@ -150,10 +150,6 @@ jobs:
150150
timeout-minutes: 15
151151
steps:
152152
- uses: actions/checkout@0ad4b8fadaa221de15dcec353f45205ec38ea70b # version 4.1.4
153-
- name: Increase ulimit
154-
if: runner.os == 'macOS'
155-
run: |
156-
ulimit -n 10000
157153
- uses: ./.github/actions/setup_node
158154
with:
159155
node-version: ${{ matrix.node }}
@@ -482,10 +478,6 @@ jobs:
482478
contents: read
483479
steps:
484480
- uses: actions/checkout@0ad4b8fadaa221de15dcec353f45205ec38ea70b # version 4.1.4
485-
- name: Increase ulimit
486-
if: runner.os == 'macOS'
487-
run: |
488-
ulimit -n 10000
489481
- name: Run e2e deployment tests
490482
uses: ./.github/actions/run_with_e2e_account
491483
with:
@@ -494,6 +486,7 @@ jobs:
494486
cdk-lib-version: ${{ needs.resolve_inputs.outputs.cdk_lib_version }}
495487
link_cli: true
496488
run: |
489+
[[ "$RUNNER_OS" == "macOS" ]] && ulimit -n 10000
497490
npm run test:dir ${{ matrix.testPaths }}
498491
e2e_generate_sandbox_tests_matrix:
499492
if: needs.do_include_e2e.outputs.run_e2e == 'true'
@@ -534,18 +527,16 @@ jobs:
534527
contents: read
535528
steps:
536529
- uses: actions/checkout@0ad4b8fadaa221de15dcec353f45205ec38ea70b # version 4.1.4
537-
- name: Increase ulimit
538-
if: runner.os == 'macOS'
539-
run: |
540-
ulimit -n 10000
541530
- name: Run e2e sandbox tests
542531
uses: ./.github/actions/run_with_e2e_account
543532
with:
544533
e2e_test_accounts: ${{ vars.E2E_TEST_ACCOUNTS }}
545534
node_version: ${{ matrix.node-version }}
546535
cdk-lib-version: ${{ needs.resolve_inputs.outputs.cdk_lib_version }}
547536
link_cli: true
548-
run: npm run test:dir ${{ matrix.testPaths }}
537+
run: |
538+
[[ "$RUNNER_OS" == "macOS" ]] && ulimit -n 10000
539+
npm run test:dir ${{ matrix.testPaths }}
549540
e2e_notices:
550541
if: needs.do_include_e2e.outputs.run_e2e == 'true'
551542
strategy:
@@ -566,10 +557,6 @@ jobs:
566557
contents: read
567558
steps:
568559
- uses: actions/checkout@0ad4b8fadaa221de15dcec353f45205ec38ea70b # version 4.1.4
569-
- name: Increase ulimit
570-
if: runner.os == 'macOS'
571-
run: |
572-
ulimit -n 10000
573560
- uses: ./.github/actions/setup_node
574561
with:
575562
node-version: 18
@@ -579,7 +566,9 @@ jobs:
579566
cdk-lib-version: ${{ needs.resolve_inputs.outputs.cdk_lib_version }}
580567
- run: cd packages/cli && npm link
581568
- name: Run e2e notices tests
582-
run: npm run test:dir packages/integration-tests/lib/test-e2e/notices.test.js
569+
run: |
570+
[[ "$RUNNER_OS" == "macOS" ]] && ulimit -n 10000
571+
npm run test:dir packages/integration-tests/lib/test-e2e/notices.test.js
583572
e2e_backend_output:
584573
if: needs.do_include_e2e.outputs.run_e2e == 'true'
585574
runs-on: ubuntu-latest
@@ -628,10 +617,6 @@ jobs:
628617
- resolve_inputs
629618
steps:
630619
- uses: actions/checkout@0ad4b8fadaa221de15dcec353f45205ec38ea70b # version 4.1.4
631-
- name: Increase ulimit
632-
if: runner.os == 'macOS'
633-
run: |
634-
ulimit -n 10000
635620
- uses: ./.github/actions/setup_node
636621
with:
637622
node-version: ${{ matrix.node-version }}
@@ -641,7 +626,9 @@ jobs:
641626
cdk-lib-version: ${{ needs.resolve_inputs.outputs.cdk_lib_version }}
642627
- run: cd packages/cli && npm link
643628
- name: Run e2e create-amplify tests
644-
run: npm run test:dir packages/integration-tests/lib/test-e2e/create_amplify.test.js
629+
run: |
630+
[[ "$RUNNER_OS" == "macOS" ]] && ulimit -n 10000
631+
npm run test:dir packages/integration-tests/lib/test-e2e/create_amplify.test.js
645632
e2e_package_manager:
646633
if: needs.do_include_e2e.outputs.run_e2e == 'true' && needs.do_include_e2e.outputs.include_package_manager_e2e == 'true'
647634
strategy:
@@ -664,10 +651,6 @@ jobs:
664651
id-token: write
665652
contents: read
666653
steps:
667-
- name: Increase ulimit
668-
if: runner.os == 'macOS'
669-
run: |
670-
ulimit -n 10000
671654
- name: Checkout aws-amplify/amplify-backend repo
672655
uses: actions/checkout@b4ffde65f46336ab88eb53be808477a3936bae11 # v4.1.1
673656
- name: Run E2E flow tests with ${{ matrix.pkg-manager }}
@@ -678,6 +661,7 @@ jobs:
678661
cdk-lib-version: ${{ needs.resolve_inputs.outputs.cdk_lib_version }}
679662
shell: bash
680663
run: |
664+
[[ "$RUNNER_OS" == "macOS" ]] && ulimit -n 10000
681665
PACKAGE_MANAGER=${{matrix.pkg-manager}} npm run test:dir packages/integration-tests/src/package_manager_sanity_checks.test.ts
682666
lint:
683667
runs-on: ubuntu-latest

0 commit comments

Comments
 (0)