Skip to content

Commit 72c2391

Browse files
authored
Enable code coverage (#9)
* Enable code coverage Signed-off-by: Charlie Truong <chtruong@nvidia.com> * Remove functional tests Signed-off-by: Charlie Truong <chtruong@nvidia.com> --------- Signed-off-by: Charlie Truong <chtruong@nvidia.com>
1 parent ba814df commit 72c2391

1 file changed

Lines changed: 8 additions & 64 deletions

File tree

.github/workflows/cicd-main.yml

Lines changed: 8 additions & 64 deletions
Original file line numberDiff line numberDiff line change
@@ -58,35 +58,10 @@ jobs:
5858
- script: L0_Tests_GPU
5959
runner: self-hosted-nemo
6060
timeout: 30
61-
needs: [cicd-container-build]
62-
runs-on: ${{ matrix.runner }}
63-
name: ${{ matrix.script }}
64-
environment: nemo-ci
65-
steps:
66-
- name: Checkout
67-
uses: actions/checkout@v4
68-
- name: main
69-
uses: ./.github/actions/test-template
70-
with:
71-
runner: ${{ runner.name }}
72-
script: ${{ matrix.script }}
73-
timeout: ${{ matrix.timeout || 10 }}
74-
is_unit_test: "true"
75-
image: emerging_optimizers
76-
cpu-only: ${{ matrix.cpu-only || false }}
77-
has-azure-credentials: "true"
78-
azure-client-id: ${{ secrets.AZURE_CLIENT_ID }}
79-
azure-tenant-id: ${{ secrets.AZURE_TENANT_ID }}
80-
azure-subscription-id: ${{ secrets.AZURE_SUBSCRIPTION_ID }}
81-
82-
cicd-unit-tests-cpu:
83-
strategy:
84-
fail-fast: false
85-
matrix:
86-
include:
8761
- script: L0_Tests_CPU
8862
runner: linux-amd64-cpu16
8963
timeout: 30
64+
cpu-only: true
9065
needs: [cicd-container-build]
9166
runs-on: ${{ matrix.runner }}
9267
name: ${{ matrix.script }}
@@ -102,35 +77,6 @@ jobs:
10277
timeout: ${{ matrix.timeout || 10 }}
10378
is_unit_test: "true"
10479
image: emerging_optimizers
105-
cpu-only: ${{ matrix.cpu-only || true }}
106-
has-azure-credentials: "true"
107-
azure-client-id: ${{ secrets.AZURE_CLIENT_ID }}
108-
azure-tenant-id: ${{ secrets.AZURE_TENANT_ID }}
109-
azure-subscription-id: ${{ secrets.AZURE_SUBSCRIPTION_ID }}
110-
111-
cicd-e2e-tests:
112-
if: false
113-
strategy:
114-
fail-fast: false
115-
matrix:
116-
include:
117-
- script: L2_Functional_Tests_GPU
118-
runner: self-hosted-nemo
119-
timeout: 30
120-
needs: [cicd-unit-tests]
121-
runs-on: ${{ matrix.runner }}
122-
name: ${{ matrix.is_optional && 'PLEASEFIXME_' || '' }}${{ matrix.script }}
123-
environment: nemo-ci
124-
steps:
125-
- name: Checkout
126-
uses: actions/checkout@v4
127-
- name: main
128-
uses: ./.github/actions/test-template
129-
with:
130-
runner: ${{ runner.name }}
131-
script: ${{ matrix.script }}
132-
timeout: ${{ matrix.timeout || 10 }}
133-
image: emerging_optimizers
13480
cpu-only: ${{ matrix.cpu-only || false }}
13581
has-azure-credentials: "true"
13682
azure-client-id: ${{ secrets.AZURE_CLIENT_ID }}
@@ -141,7 +87,6 @@ jobs:
14187
needs:
14288
- cicd-container-build
14389
- cicd-unit-tests
144-
- cicd-e2e-tests
14590
if: always()
14691
runs-on: ubuntu-latest
14792
permissions: write-all
@@ -240,7 +185,7 @@ jobs:
240185
needs: [Nemo_CICD_Test]
241186
strategy:
242187
matrix:
243-
flag: [unit-test, e2e]
188+
flag: [unit-test]
244189
steps:
245190
- name: Checkout
246191
uses: actions/checkout@v4
@@ -263,13 +208,12 @@ jobs:
263208
rm -rf coverage-*
264209
ls -al
265210
266-
# Disabled for new repos initially
267-
# - name: Upload coverage reports to Codecov
268-
# uses: codecov/codecov-action@v5
269-
# with:
270-
# token: ${{ secrets.CODECOV_TOKEN }}
271-
# verbose: true
272-
# flags: ${{ matrix.flag }}
211+
- name: Upload coverage reports to Codecov
212+
uses: codecov/codecov-action@v5
213+
with:
214+
token: ${{ secrets.CODECOV_TOKEN }}
215+
verbose: true
216+
flags: ${{ matrix.flag }}
273217

274218
- name: Upload artifacts
275219
uses: actions/upload-artifact@v4

0 commit comments

Comments
 (0)