Skip to content

Fix standalone test for direct entrypoints - #862

Open
PetrNovota wants to merge 9 commits into
feature/luxonis_apps_release_workflowfrom
codex/fix-standalone-entrypoint-test
Open

Fix standalone test for direct entrypoints#862
PetrNovota wants to merge 9 commits into
feature/luxonis_apps_release_workflowfrom
codex/fix-standalone-entrypoint-test

Conversation

@PetrNovota

Copy link
Copy Markdown
Contributor

Summary

  • Keep oakctl app run attached while direct-entrypoint apps are monitored.
  • Drain app output concurrently instead of closing stdout and waiting immediately.
  • Enforce startup/runtime failures and guarantee subprocess cleanup.

Root cause

The standalone harness closed the oakctl output stream and called process.wait() as soon as it saw App output:. Apps launched directly by /entrypoint.sh keep oakctl app run attached for the application lifetime, so the test blocked until the outer CI timeout.

This change retains /entrypoint.sh as the app launcher; runit remains limited to base-image helper services.

Validation

  • Targeted standalone test against an OAK4-D R9: 1 passed in 170.22s.
  • App reached Pipeline created. and remained running for the configured health interval.
  • ruff and ruff-format hooks passed.
  • python3 -m py_compile tests/test_examples_standalone.py.
  • git diff --check.

@PetrNovota
PetrNovota force-pushed the feature/luxonis_apps_release_workflow branch 2 times, most recently from 60f553e to b6d090b Compare September 10, 2026 09:19
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.

1 participant