Skip to content

LLT-7587: natlab: wait for starting containers before releasing control - #1968

Merged
gytsto merged 1 commit into
mainfrom
natlab-wait-starting-containers
Aug 27, 2026
Merged

LLT-7587: natlab: wait for starting containers before releasing control#1968
gytsto merged 1 commit into
mainfrom
natlab-wait-starting-containers

Conversation

@gytsto

@gytsto gytsto commented Aug 25, 2026

Copy link
Copy Markdown
Contributor

Problem

A container crashing during compose up --wait aborts the whole wait early. manage_containers() recovery only restarted missing/unhealthy services and treated health starting as fine, so tests started while slow-boot VMs were still booting. Setup checks then hit a half-booted VM (SSH connection refused) and killed the session (libtelio-build job 38345856 on v6.1.4: openwrt-gw-03 crashed once, Windows VM got ~2.5 min instead of ~10).

Solution

manage_containers() now also tracks starting services and polls until they resolve to healthy/unhealthy, with a 1200s backstop (every healthcheck resolves starting within ~300s by config). starting flipping to unhealthy still goes through the existing restart path.

☑️ Definition of Done checklist

  • Commit history is clean (requirements)
  • README.md is updated
  • Functionality is covered by unit or integration tests

@gytsto
gytsto requested a review from a team as a code owner August 25, 2026 13:12
@gytsto
gytsto deployed to Internal August 25, 2026 13:12 — with GitHub Actions Active
@gytsto gytsto changed the title natlab: wait for starting containers before releasing control LLT-7587: natlab: wait for starting containers before releasing control Aug 26, 2026
Comment thread nat-lab/natlab.py Outdated
Comment thread nat-lab/natlab.py
@gytsto
gytsto force-pushed the natlab-wait-starting-containers branch from e88f1f1 to 0f528d5 Compare August 26, 2026 12:14
@gytsto
gytsto deployed to Internal August 26, 2026 12:16 — with GitHub Actions Active
@gytsto
gytsto force-pushed the natlab-wait-starting-containers branch from 0f528d5 to e88f1f1 Compare August 26, 2026 12:20
@gytsto
gytsto deployed to Internal August 26, 2026 12:23 — with GitHub Actions Active
When a container crashes during 'compose up --wait', compose aborts the
whole wait early and manage_containers() only restarted missing/unhealthy
services, letting tests start while slow-boot VMs were still in the
'starting' health state. Setup checks then hit a half-booted VM and
failed the session.
@gytsto
gytsto force-pushed the natlab-wait-starting-containers branch from e88f1f1 to 38e3aa3 Compare August 26, 2026 12:31

@tomaszklak tomaszklak left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1

@Paintree Paintree left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1

@gytsto
gytsto added this pull request to the merge queue Aug 27, 2026
Merged via the queue into main with commit f8e300e Aug 27, 2026
65 checks passed
@gytsto
gytsto deleted the natlab-wait-starting-containers branch August 27, 2026 07:10
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