NVIDIA / gpu-operator
38%
main: 38%

LAST BUILD BRANCH: pull-request/2750
DEFAULT BRANCH: main
Repo Added
Files 73
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

LAST BUILD ON BRANCH pull-request/2718
branch: pull-request/2718
CHANGE BRANCH
x

coverage: 38.385%. Remained the same
31073033892

push

github

abrarshivani 
refactor(ci): tidy the e2e test scripts

Store a deadline rather than a start time in the polling loops. Comparing
SECONDS against a deadline computed once says what the loop means without the
subtraction, and it removes the comment each loop was carrying to explain that
SECONDS counts from shell start rather than from the loop.

migrate-clusterpolicy-to-nvidiadriver.sh had six loops still counting sleeps
instead of measuring time, the same bug already fixed in checks.sh and
update-nvidiadriver.sh. It runs on the containerd path, so convert those too.

check_gpu_pod_ready listed every pod in the cluster as json on every pass of a
five second loop, but only used the result when it regenerated the log files
every thirty seconds. Fetch it where it is used, ask for two custom columns
instead of the whole object, and read it with the shell rather than a jq
invocation per pod. The same loop printed the pod table once for the console
and fetched it again for the log file, which tee does in one call.

The request timeout default now lives in .definitions.sh with the other
defaults instead of being repeated at each use.

Also guard five more diagnostic calls that the earlier pass missed, replace
the runtime comparison in node-operations.sh with a case statement, and drop
the argument and readability checks in node-exec.sh that only repeat what
node-operations.sh and the redirect already report.

Signed-off-by: Abrar Shivani <ashivani@nvidia.com>

5572 of 14516 relevant lines covered (38.39%)

0.44 hits per line

Relevant lines Covered
14516 RELEVANT LINES 5572 COVERED LINES
0.44 HITS PER LINE
Source Files on pull-request/2718

Recent builds

Builds Branch Commit Type Ran Committer Via Coverage
31073033892 pull-request/2718 refactor(ci): tidy the e2e test scripts Store a deadline rather than a start time in the polling loops. Comparing SECONDS against a deadline computed once says what the loop means without the subtraction, and it removes the comment each loop was ... push abrarshivani github
38.39
31064334768 pull-request/2718 fix(ci): survive a transient API outage during the driver upgrade Both e2e jobs died in wait_for_driver_upgrade_done. The driver upgrade restarts the container runtime on the node, and now that the tests drive the cluster from the runner rather t... push abrarshivani github
38.39
31059357931 pull-request/2718 fix(ci): upload e2e logs on cancelled runs too The archive step was gated on failure(), but a job stopped by timeout-minutes is cancelled rather than failed, so failure() evaluates false and the upload is skipped. That loses the logs on exactly t... push abrarshivani github
38.39
See All Builds (2342)