Skip to content

Fix Vertex batch live tests - #538

Merged
aksg87 merged 1 commit into
mainfrom
aksg87/fix-batch-live-tests
Sep 13, 2026
Merged

aksg87 merged 1 commit into
mainfrom
aksg87/fix-batch-live-tests

Conversation

@aksg87

@aksg87 aksg87 commented Sep 13, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Supply the required Vertex batch caching and retention settings without changing shared-bucket lifecycle rules.
  • Use the supported US multi-region in Vertex live tests and the batch guide; keep VERTEX_LOCATION overridable.
  • Check the current schema_config argument passed to batch inference.
  • Verify cache reuse with identical grounded extractions and no second batch submission, replacing timing checks and bucket-wide cache scans.

No runtime code or dependency changes.

Verification

  • pytest tests --import-mode=importlib -q -m 'not live_api and not requires_pip and not integration': 838 passed against the installed release-candidate wheel.
  • tox -e format,lint-src,lint-tests: passed.
  • Non-batch live API suite: 12 passed.
  • OpenAI Batch live test: passed with two synthetic documents; uploaded and output files cleaned up.
  • Vertex batch extraction/cache live tests: 2 passed, including schema forwarding, grounded medication/dosage results, and cache reuse without another submission.

Live validation used fresh test buckets; test objects and buckets were cleaned up. The repository's shared bucket and its lifecycle policy were not modified.

@github-actions github-actions Bot added the size/S Pull request with 50-150 lines changed label Sep 13, 2026
@aksg87
aksg87 merged commit 2b3668b into main Sep 13, 2026
19 checks passed
@aksg87 aksg87 mentioned this pull request Sep 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/S Pull request with 50-150 lines changed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant