Skip to content

Commit b9d47ef

Browse files
ivanvck8s-infra-cherrypick-robot
authored andcommitted
make: upload partial coverage reports
In the case of workflow failures (i.e., flaky tests), we still should upload a partial coverage report to avoid lagging reports on the stable branches. Signed-off-by: Ivan Valdes <[email protected]>
1 parent 5a9094c commit b9d47ef

File tree

1 file changed

+5
-2
lines changed

1 file changed

+5
-2
lines changed

Makefile

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -63,8 +63,11 @@ test-coverage:
6363
COVERDIR=covdir PASSES="build cov" ./scripts/test.sh $(GO_TEST_FLAGS)
6464

6565
.PHONY: upload-coverage-report
66-
upload-coverage-report: test-coverage
67-
COVERDIR=covdir ./scripts/codecov_upload.sh
66+
upload-coverage-report:
67+
return_code=0; \
68+
$(MAKE) test-coverage || return_code=$$?; \
69+
COVERDIR=covdir ./scripts/codecov_upload.sh; \
70+
exit $$return_code
6871

6972
.PHONY: fuzz
7073
fuzz:

0 commit comments

Comments
 (0)