Skip to content

Commit

Permalink
update core-crucible-ci to make the primary jobs timeout-minutes valu…
Browse files Browse the repository at this point in the history
…e dynamic and dependent on whether or not a controller build is being used

- when a controller build is being used the timeout-minutes needs to
  be higher since more image builds are required and they make the
  jobs run longer
  • Loading branch information
k-rister committed Aug 28, 2024
1 parent 19db52b commit 10faa9d
Showing 1 changed file with 16 additions and 2 deletions.
18 changes: 16 additions & 2 deletions .github/workflows/core-crucible-ci.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -63,6 +63,7 @@ jobs:
self_hosted_scenarios: ${{ steps.get-scenarios-self.outputs.scenarios }}
repo_name: ${{ steps.get-repo-name.outputs.repo-name }}
ci_target_dir: ${{ steps.set-ci-target-dir.outputs.ci-target-dir }}
timeout_minutes: ${{ steps.determine-timeout-minutes.outputs.timeout-minutes }}
steps:

- name: checkout crucible default
Expand Down Expand Up @@ -174,6 +175,19 @@ jobs:
echo "ci-target-dir=${{ inputs.ci_target_dir }}" >> $GITHUB_OUTPUT
fi
- name: determine timeout-minutes
id: determine-timeout-minutes
run: |
if [ "${{ steps.check-controller-build.outputs.build-controller }}" == "yes" ]; then
build_controller_timeout_minutes=150
echo "Setting timeout-minutes to the build controller value of '${build_controller_timeout_minutes}'"
echo "timeout-minutes=${build_controller_timeout_minutes}" >> $GITHUB_OUTPUT
else
no_build_controller_timeout_minutes=90
echo "Setting timeout-minutes to the no build controller value of '${no_build_controller_timeout_minutes}'"
echo "timeout-minutes=${no_build_controller_timeout_minutes}" >> $GITHUB_OUTPUT
fi
display-params:
runs-on: [ self-hosted, workflow-overhead ]
timeout-minutes: 10
Expand Down Expand Up @@ -260,7 +274,7 @@ jobs:

github-runners:
runs-on: ubuntu-latest
timeout-minutes: 120
timeout-minutes: ${{ needs.gen-params.outputs.timeout_minutes }}
needs:
- gen-params
- display-params
Expand Down Expand Up @@ -337,7 +351,7 @@ jobs:

self-hosted-runners:
runs-on: [ self-hosted, cpu-partitioning, remotehost, remotehosts ]
timeout-minutes: 120
timeout-minutes: ${{ needs.gen-params.outputs.timeout_minutes }}
needs:
- gen-params
- display-params
Expand Down

0 comments on commit 10faa9d

Please sign in to comment.