[kellyguo11]

Description

Enable the root ov extra in the release base Docker image so its locked environment includes ovphysx, ovrtx, and ovstage. This aligns release/3.0.0 with the base-image dependency set on develop and fixes OVRTX-backed task configurations failing during standard Docker package tests with ModuleNotFoundError: No module named 'ovrtx'.\n\nThis is a focused release-branch workflow fix for the CI failure exposed by #8318. It does not change dependency pins or the lockfile.\n\n## Type of change\n\n- Bug fix (non-breaking Docker dependency fix)\n\n## Release backport\n\nNot applicable; this PR targets release/3.0.0 directly.\n\n## Screenshots\n\nNot applicable.\n\n## Validation\n\n- uv run --extra test python -m pytest docker/test/test_dockerfile_nonroot.py — 24 passed\n- SKIP=check-changelog-fragments uv run isaaclab -f — all applicable formatting and lint hooks passed\n- git diff --check\n\nThe full formatting command’s changelog hook compares the release snapshot against develop and reports the release branch’s already-compiled fragments. No source/<package> files are changed, so no package changelog fragment applies.\n\n## Checklist\n\n- [x] I have read and understood the contribution guidelines\n- [x] I have run the applicable formatting and lint checks\n- [x] Documentation changes are not required for this Docker dependency fix\n- [x] My changes generate no new warnings\n- [x] I have validated the closest Docker contract tests\n- [x] No source package changed, so no package changelog fragment is required\n- [x] My name already exists in CONTRIBUTORS.md

[kellyguo11]

run-ci

[isaaclab-review-bot]

Isaac Lab Review Bot

All reviewed code categories look good.

• PR description: The scope matches the diff, but literal \n sequences prevent most body sections from rendering; replace them with actual line breaks.
• Design and architecture: Architecture looks good.
• Scope and over-engineering: No over-engineering issues.
• API: API changes look good.
• Compatibility and deprecation: No breaking changes identified.
• Implementation: Implementation looks good.
• Style consistency: Style consistency looks good.
• Test quality: Test quality looks good.

No blocking issues. No inline findings.

Automated review; human maintainers own approval decisions.

[greptile-apps]

Confidence Score: 4/5

[High risk] Adds a new optional dependency to the base Docker image.

The PR appears safe to merge, with a non-blocking test-coverage gap for the new base-image dependency contract.

Findings

1.  Base image dependency untested ▶

Summary

The PR adds the root ov extra to the release base Docker image’s default dependency set, making its locked OV runtime packages available to Docker tasks.

• The existing Dockerfile contract tests do not assert that the base image selects those packages.

_{Reviews (1) · Last reviewed commit: "Enable OV runtime in release base Docker..."}

[greptile-apps]

Base image dependency untested The new --extra ov makes ovphysx, ovrtx, and ovstage part of the default base image, but the Dockerfile contract tests check package installation only for the kitless image. Please add a base-image assertion for this extra or its packages; otherwise, a later edit could silently remove this fix without failing those tests.

[kellyguo11]

run-ci

[ooctipus]

run-ci

[ooctipus]

run-ci