Skip to content

Commit

Permalink
Fix unit test CI check failures
Browse files Browse the repository at this point in the history
The ec2 instance used to run the unit tests cannot launch because of a configuration issue.

Signed-off-by: Courtney Pacheco <[email protected]>
  • Loading branch information
courtneypacheco committed Jan 16, 2025
1 parent 989e4fb commit 413e97e
Showing 1 changed file with 9 additions and 2 deletions.
11 changes: 9 additions & 2 deletions .github/workflows/unittesting-ci-nvidia.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -13,9 +13,11 @@ on:
pull_request:
types: [opened, reopened, synchronize]
push:
# TEMPORARILY ADDING TESTING BRANCH. WILL REMOVE.
branches:
- "main"
- "release-**"
- "fix-unit-test-workflow-file"

env:
pytest_mark: "fast"
Expand Down Expand Up @@ -48,7 +50,9 @@ jobs:
mode: start
github-token: ${{ secrets.GH_PERSONAL_ACCESS_TOKEN }}
ec2-image-id: ${{ vars.AWS_EC2_AMI }}
ec2-instance-type: ${{ vars.AWS_REGION }}
# TODO: Update the EC2 instance type to use the EC2 runner variant when GPU calls are mocked
# ec2-instance-type: ${{ env.ec2_runner_variant }}
ec2-instance-type: g4dn.12xlarge
subnet-id: subnet-024298cefa3bedd61
security-group-id: sg-06300447c4a5fbef3
iam-role-name: instructlab-ci-runner
Expand Down Expand Up @@ -104,6 +108,7 @@ jobs:
- name: "Run unit tests with Tox and Pytest"
run: |
. venv/bin/activate
tox -e py3-unit -- -m ${{env.pytest_mark}}
- name: "Show disk utilization AFTER tests"
Expand Down Expand Up @@ -134,4 +139,6 @@ jobs:
mode: stop
github-token: ${{ secrets.GH_PERSONAL_ACCESS_TOKEN }}
label: ${{ needs.start-ec2-runner.outputs.label }}
ec2-instance-type: ${{ env.ec2_runner_variant }}
ec2-instance-id: ${{ needs.start-ec2-runner.outputs.ec2-instance-id }}
# TODO: Update the EC2 instance type to use the EC2 runner variant when GPU calls are mocked
# ec2-instance-type: ${{ env.ec2_runner_variant }}

0 comments on commit 413e97e

Please sign in to comment.