Skip to content

perf(ci): shorten unstructured-api test and publishing runs - #587

Merged
CyMule merged 1 commit into
mainfrom
perf/api-ci-runtime
Sep 11, 2026
Merged

CyMule merged 1 commit into
mainfrom
perf/api-ci-runtime

Conversation

@CyMule

@CyMule CyMule commented Sep 10, 2026 •

Copy link
Copy Markdown
Contributor

What

Standalone API CI avoids repeated inference work and redundant image transfers while retaining distinct smoke-test scenarios.

Measurement Before After Saved
Total CI 39m18s historical median 26m15s 13m03s (33.2%)
Smoke tests 16m05s 11m37s 4m28s (27.8%)
Mode comparisons ~14m35s 5m07s ~9m28s (64.9%)

Successful PR run. Total baseline is nine historical runs; test-stage baselines come from August 27. This is not a controlled comparison: runner variability and faster individual inference calls contribute to the difference.

Why

Some option checks unintentionally repeated default inference, gzip parametrization duplicated seven cases, and the smoke-test client installed unused dependencies. The test harness now sends the intended options and avoids duplicated work. Release publishing also retags images in the registry instead of pulling them only to retag them.

The 37 distinct smoke cases and seven mode comparisons remain covered. Heavy PDF inference remains the largest test cost.

Relationship to other work

Independent of Plugins, platform-libs, and shared-workflow #434/#437. Core-product has its own vendored API and separate CI improvement in #1660; its savings are not an additional stage of this pipeline.

Publishing speed has not been measured: that workflow runs only on main. The timing table covers CI, not release publishing.

Validation

Current GitHub checks pass, including the Docker build, 37 smoke tests, seven mode comparisons, core tests, coverage, and lint. Six shell regression tests and registry-retagging dry runs passed; publishing itself still requires validation on main.

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

No issues found across 8 files

Shadow auto-approve: would not auto-approve. This PR does not meet the repository auto-approval settings.

Re-trigger cubic

@CyMule CyMule changed the title perf(ci): reduce redundant inference and image transfers perf(ci): shorten unstructured-api test and publishing runs Sep 10, 2026
@CyMule
CyMule merged commit cba716b into main Sep 11, 2026
15 checks passed
@CyMule
CyMule deleted the perf/api-ci-runtime branch September 11, 2026 14:43
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.

2 participants