Skip to content

CI: Prevent zombie MAD RTE perftest server#11630

Open
Alexey-Rivkin wants to merge 1 commit into
openucx:masterfrom
Alexey-Rivkin:ci-mad-test-hardening
Open

CI: Prevent zombie MAD RTE perftest server#11630
Alexey-Rivkin wants to merge 1 commit into
openucx:masterfrom
Alexey-Rivkin:ci-mad-test-hardening

Conversation

@Alexey-Rivkin

@Alexey-Rivkin Alexey-Rivkin commented Jul 7, 2026

Copy link
Copy Markdown
Contributor

What?

Stop the MAD perftest server container reliably, and clear any zombie ucx_perftest_* container before starting a new one.

Why?

On a failed run, docker_stop_srv runs cat on a missing log, which aborts under set -e before docker stop. The detached server survives as a zombie and holds the MAD registration on the HCA, so every later build's server dies with mad_register_server_via ... Invalid argument and the client hangs until timeout. Broke all PRs until zombies were cleared by hand (e.g. build 127843).

How?

  • docker_stop_srv: docker stop before cat.
  • docker_run_srv: docker rm -f sweep of ucx_perftest_* to heal zombies from a canceled or crashed prior run.

@Alexey-Rivkin Alexey-Rivkin force-pushed the ci-mad-test-hardening branch from 3d49ad0 to e942c04 Compare July 7, 2026 15:49
@Alexey-Rivkin Alexey-Rivkin changed the title CI: Make MAD RTE perftest teardown leak-proof CI: Stop MAD perftest server before printing its log Jul 7, 2026
@Alexey-Rivkin Alexey-Rivkin force-pushed the ci-mad-test-hardening branch 2 times, most recently from d949803 to bd1126d Compare July 7, 2026 16:49
@Alexey-Rivkin Alexey-Rivkin changed the title CI: Stop MAD perftest server before printing its log CI: Prevent zombie MAD perftest server from wedging the stage Jul 7, 2026
@Alexey-Rivkin Alexey-Rivkin changed the title CI: Prevent zombie MAD perftest server from wedging the stage CI: Prevent zombie MAD RTE perftest server Jul 7, 2026
A failed run leaves behind a zombie server container: docker_stop_srv
cats the log before docker stop, and when the log is missing the cat
aborts the script (set -e) before the stop runs. Being a detached
container, the zombie survives the build and keeps the MAD agent
registration on the HCA, so every following build's server fails with
EINVAL and the client hangs until the job timeout - the stage breaks
for all PRs until the zombie is removed by hand.

Stop the container first and sweep zombie ucx_perftest_* containers
before starting a new server, so one bad run cannot wedge the stage.
@Alexey-Rivkin Alexey-Rivkin force-pushed the ci-mad-test-hardening branch from bd1126d to 0f13015 Compare July 7, 2026 16:54
@Alexey-Rivkin Alexey-Rivkin marked this pull request as ready for review July 7, 2026 17:05
@Alexey-Rivkin Alexey-Rivkin requested a review from dpressle July 7, 2026 17:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants