Skip to content

Commit 966f6b9

Browse files
committed
ci: drop nydus-unit-test-coverage
There is no need to run the nydus unit tests twice, once in nydus-unit-test-coverage, the other in nydus-unit-test. Signed-off-by: Peng Tao <bergwolf@hyper.sh>
1 parent ba325ff commit 966f6b9

File tree

1 file changed

+9
-29
lines changed

1 file changed

+9
-29
lines changed

.github/workflows/smoke.yml

Lines changed: 9 additions & 29 deletions
Original file line numberDiff line numberDiff line change
@@ -248,8 +248,15 @@ jobs:
248248
- name: Unit Test
249249
run: |
250250
sudo -E ENABLE_DEBUG=1 "PATH=$PATH" make ut-nextest
251+
sudo -E "PATH=$PATH" make generate-codecov
252+
- name: Upload nydus coverage file
253+
uses: actions/upload-artifact@v6
254+
with:
255+
name: nydus-test-coverage-artifact
256+
path: |
257+
coverage/coverage.info
251258
252-
contrib-unit-test-coverage:
259+
contrib-unit-test:
253260
runs-on: ubuntu-latest
254261
steps:
255262
- name: Checkout
@@ -269,36 +276,9 @@ jobs:
269276
path: |
270277
contrib/nydusify/coverage.txt
271278
272-
nydus-unit-test-coverage:
273-
runs-on: ubuntu-latest
274-
env:
275-
CARGO_TERM_COLOR: always
276-
steps:
277-
- uses: actions/checkout@v6
278-
- name: Rust Cache
279-
uses: Swatinem/rust-cache@v2
280-
with:
281-
cache-on-failure: true
282-
shared-key: Linux-cargo-amd64
283-
save-if: ${{ github.ref == 'refs/heads/master' }}
284-
- name: Install cargo-llvm-cov
285-
uses: taiki-e/install-action@cargo-llvm-cov
286-
- name: Fscache Setup
287-
run: sudo bash misc/fscache/setup.sh
288-
- name: Generate code coverage
289-
run: |
290-
sudo -E ENABLE_DEBUG=1 "PATH=$PATH" make ut
291-
sudo -E "PATH=$PATH" make generate-codecov
292-
- name: Upload nydus coverage file
293-
uses: actions/upload-artifact@v6
294-
with:
295-
name: nydus-test-coverage-artifact
296-
path: |
297-
coverage/coverage.info
298-
299279
upload-coverage-to-codecov:
300280
runs-on: ubuntu-latest
301-
needs: [contrib-unit-test-coverage, nydus-unit-test-coverage]
281+
needs: [contrib-unit-test, nydus-unit-test]
302282
steps:
303283
- uses: actions/checkout@v6
304284
- name: Download nydus coverage file

0 commit comments

Comments
 (0)