Add app log stream affinity for scalable apps - #6742
jerrylian-db wants to merge 4 commits into
Conversation
Approval status: pending
|
Integration test reportCommit: 63565f6
Top 3 slowest tests (at least 2 minutes):
|
pkosiec
left a comment
There was a problem hiding this comment.
Overal LGTM, nice work 👍
Are there any plans to allow users to connect to a specific replica with a flag? That would be awesome.
|
An authorized user can trigger integration tests manually by following the instructions below: Trigger: Inputs:
Checks will be approved automatically on success. |
Changes
databricks apps logsis used with an app configured for more than one instance.__Host-databricks-app-routercookie on log websocket handshakes for scalable apps.Why
databricks apps logs --followreconnects the websocket on transient failures. A stable routing cookie keeps reconnects best-effort affined to the same app instance, matching scalable Apps session affinity guidance.Tests
go test ./cmd/apps ./libs/apps/logstream./task checks./task lint-q(Go lint passed; Pythonrufffetch failed because PyPI DNS resolution failed)./task test(unit test phases passed; acceptance phase blocked because localjqis 1.6 and the suite requiresjq >= 1.7)This PR was written by Codex.