Skip to content

fix(tei): retry embedding connect timeouts - #3097

Merged
nicoloboschi merged 1 commit into
vectorize-io:mainfrom
jooneyp:fix/tei-connect-timeout-retry
Aug 4, 2026
Merged

fix(tei): retry embedding connect timeouts#3097
nicoloboschi merged 1 commit into
vectorize-io:mainfrom
jooneyp:fix/tei-connect-timeout-retry

Conversation

@jooneyp

@jooneyp jooneyp commented Jul 31, 2026

Copy link
Copy Markdown
Contributor

Summary

  • retry httpx.ConnectTimeout in RemoteTEIEmbeddings using the existing bounded retry budget
  • add focused coverage for transient recovery and persistent timeout exhaustion

Test plan

  • RED: both new ConnectTimeout regressions failed before the production change
  • cd hindsight-api-slim && uv run pytest tests/test_tei_embeddings.py -q (5 passed)
  • ./scripts/hooks/lint.sh
  • independent read-only code review (no findings)

No schema, auth, public API, or deployment changes.

@nicoloboschi nicoloboschi left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

lgtm

@nicoloboschi
nicoloboschi merged commit 417efb3 into vectorize-io:main Aug 4, 2026
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