fix(proxy): keep hard continuity owners ahead of affinity - #1732
Conversation
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 2d112a6e35
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
2d112a6 to
44d6c2f
Compare
Komzpa
changed the title
4c84dd6 to
5eb9304
Compare
|
@codex review |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 5eb930427e
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
5eb9304 to
2c5f314
Compare
|
@codex review |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 2c5f31443d
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
…32-20260814 # Conflicts: # app/modules/usage/live_ingest.py
github-actions
Bot
added
the
db migration
Komzpa
added
needs rebase
1 participant
Summary
Validation
uv run pytest tests/unit/test_proxy_utils.py::test_plain_stream_resolves_http_bridge_turn_state_owner tests/unit/test_live_usage_ingest.py::test_ingestor_resolves_upstream_account_id_before_history_write tests/unit/test_live_usage_ingest.py::test_stream_responses_tap_publishes_rate_limit_eventsuv run pytest tests/unit/test_proxy_utils.py::test_compact_turn_state_owner_never_falls_back_to_unscoped_sticky_sessions tests/unit/test_proxy_utils.py::test_compact_turn_state_owner_is_a_strict_selection_constraint tests/unit/test_proxy_utils.py::test_plain_stream_resolves_http_bridge_turn_state_owner tests/unit/test_proxy_utils.py::test_compact_file_pin_overrides_session_and_prompt_cache_locality tests/unit/test_live_usage_ingest.py::test_ingestor_resolves_upstream_account_id_before_history_write tests/unit/test_live_usage_ingest.py::test_stream_responses_tap_publishes_rate_limit_eventsgit diff --checkLive note
Initial live recovery at
codex-lb:live-stack-790c93e/790c93ef2f0b0d94aac1fb72b0f249304271fb6bremoved the compact owner-alias failure, but a later live image-capable stream still hitcontinuity_owner_conflictafter bypassing the HTTP bridge and live usage ingest loggedsqlite3.IntegrityError: FOREIGN KEY constraint failed. This PR now includes those follow-up fixes; live redeploy/readback will be tracked separately from PR CI.