Skip to content

fix(pipeline): close cloud E2E runtime gaps - #1502

Open
Alexandre Gattiker (algattik) wants to merge 1 commit into
mainfrom
test/1490-cloud-e2e-runtime-gaps
Open

Alexandre Gattiker (algattik) wants to merge 1 commit into
mainfrom
test/1490-cloud-e2e-runtime-gaps

Conversation

@algattik

@algattik Alexandre Gattiker (algattik) commented Sep 9, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • harden Azure ML and OSMO cloud E2E runtime validation and cleanup
  • validate MLflow provenance, model registration, and runtime dependency state
  • align OSMO LeRobot submission arguments with the current CLI

Validation

  • all 10 cloud E2E test modules passed against the deployed environment
  • npm run lint:py
  • uvx --with ruff==0.15.21 ruff format --check .
  • npm run spell-check
  • env -u UV_DEFAULT_INDEX uv lock --check --offline
  • PR validation summary passed

Closes #1490

@github-actions

github-actions Bot commented Sep 9, 2026

Copy link
Copy Markdown
Contributor

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Snapshot Warnings

⚠️: No snapshots were found for the head SHA 6d4fb0c.
Ensure that dependencies are being submitted on PR branches and consider enabling retry-on-snapshot-warnings. See the documentation for more information and troubleshooting advice.

Scanned Files

None

@codecov-commenter

Codecov Comments Bot (codecov-commenter) commented Sep 9, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 87.86127% with 21 lines in your changes missing coverage. Please review.
✅ Project coverage is 88.12%. Comparing base (e3b2b92) to head (6d4fb0c).

Files with missing lines Patch % Lines
...ment/viewer/frontend/src/hooks/use-joint-config.ts 60.00% 4 Missing and 2 partials ⚠️
...rc/components/ai-suggestions/AISuggestionPanel.tsx 25.00% 3 Missing ⚠️
...management/viewer/frontend/src/hooks/use-labels.ts 86.36% 1 Missing and 2 partials ⚠️
...on-workspace/useAnnotationWorkspaceMediaSources.ts 33.33% 2 Missing ⚠️
data-management/viewer/frontend/src/api/export.ts 93.75% 0 Missing and 1 partial ⚠️
...end/src/components/annotation-panel/LabelPanel.tsx 66.66% 0 Missing and 1 partial ⚠️
...ponents/annotation-panel/ObjectDetectionWidget.tsx 75.00% 1 Missing ⚠️
...omponents/episode-analyzer/EpisodeAnalysisCard.tsx 80.00% 0 Missing and 1 partial ⚠️
...components/episode-analyzer/MotionMetricsPanel.tsx 0.00% 0 Missing and 1 partial ⚠️
...a-management/viewer/frontend/src/lib/api-client.ts 97.87% 0 Missing and 1 partial ⚠️
... and 1 more

❌ Your patch status has failed because the patch coverage (77.50%) is below the target coverage (80.00%). You can increase the patch coverage or adjust the target coverage.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1502      +/-   ##
==========================================
+ Coverage   88.11%   88.12%   +0.01%     
==========================================
  Files         279      279              
  Lines       23289    23250      -39     
  Branches     3214     3154      -60     
==========================================
- Hits        20520    20490      -30     
+ Misses       2079     2071       -8     
+ Partials      690      689       -1     
Flag Coverage Δ *Carryforward flag
go 100.00% <ø> (ø)
pester 86.75% <ø> (ø) Carriedforward from e3b2b92
pytest-data-pipeline 100.00% <ø> (ø) Carriedforward from e3b2b92
pytest-dataviewer 89.66% <100.00%> (-0.06%) ⬇️
pytest-dm-tools 100.00% <ø> (ø) Carriedforward from e3b2b92
pytest-evaluation 95.40% <ø> (ø)
pytest-fuzz 3.98% <0.00%> (+<0.01%) ⬆️
pytest-inference 100.00% <ø> (ø) Carriedforward from e3b2b92
pytest-shared-ci 100.00% <ø> (ø) Carriedforward from e3b2b92
pytest-training 92.58% <80.00%> (-0.03%) ⬇️
vitest 85.66% <87.57%> (+0.10%) ⬆️
vitest-app 85.66% <87.57%> (+0.10%) ⬆️
vitest-components 85.66% <87.57%> (+0.10%) ⬆️
vitest-features 85.66% <87.57%> (+0.10%) ⬆️
vitest-lib 85.66% <87.57%> (+0.10%) ⬆️
vitest-state 85.66% <87.57%> (+0.10%) ⬆️

*This pull request uses carry forward flags. Click here to find out more.

Files with missing lines Coverage Δ
...anagement/viewer/backend/src/api/routers/export.py 90.68% <100.00%> (-0.55%) ⬇️
...-management/viewer/frontend/src/api/ai-analysis.ts 100.00% <100.00%> (ø)
...ta-management/viewer/frontend/src/api/detection.ts 100.00% <100.00%> (+5.26%) ⬆️
...d/src/components/ai-suggestions/SuggestionCard.tsx 94.73% <100.00%> (ø)
...omponents/annotation-workspace/useFramePrefetch.ts 100.00% <100.00%> (ø)
.../src/components/curriculum/CurriculumGenerator.tsx 98.07% <100.00%> (ø)
...nd/src/components/curriculum/CurriculumPreview.tsx 100.00% <100.00%> (ø)
...frontend/src/components/dashboard/ActivityFeed.tsx 100.00% <ø> (ø)
.../src/components/dashboard/AnnotatorLeaderboard.tsx 95.23% <100.00%> (ø)
...tend/src/components/dashboard/QualityDashboard.tsx 100.00% <ø> (ø)
... and 15 more

... and 1 file with indirect coverage changes

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

- align cloud E2E runtime behavior with current submission interfaces
- remove obsolete OSMO IL submission arguments
- satisfy Python and spelling checks

🛠️ - Generated by Copilot

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot-Session: 3f58b3fa-a69e-47a8-a38f-a33a5da257c5
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.

test(training): close cloud E2E runtime validation gaps

2 participants