Skip to content

feat(web): add per-container log filtering to the log pane#6763

Open
breezy-devs wants to merge 2 commits into
tilt-dev:masterfrom
breezy-devs:container-log-filter
Open

feat(web): add per-container log filtering to the log pane#6763
breezy-devs wants to merge 2 commits into
tilt-dev:masterfrom
breezy-devs:container-log-filter

Conversation

@breezy-devs
Copy link
Copy Markdown

@breezy-devs breezy-devs commented May 9, 2026

Closes #3814

Summary

  • Adds a container structured field to every pod log line in the streaming controller, enabling frontend filtering without text parsing
  • Adds a container filter dropdown to the log pane action bar, shown when a resource has logs from 1+ distinct containers: disabled at exactly 1 (layout stays stable once any container logs arrive), interactive at 2+
  • Resources with no container logs (e.g. local_resource) never render the dropdown
  • Filter selection is URL-persisted via the existing search param pattern and integrates with the existing FilterSet pipeline

Multi-Container

image

Single Container

image

No container

image

Test plan

  • Run go test ./internal/controllers/core/podlogstream/... to verify the new TestLogContainerField test passes
  • Run yarn test in web/ to verify all frontend tests pass
  • Deploy a multi-container resource (e.g. app + sidecar) and confirm the container dropdown appears and is interactive
  • Verify selecting a container filters logs correctly and the selection round-trips through the URL
  • Verify a single-container resource shows the dropdown disabled with the container name in the right pill
  • Verify a resource with no container logs (e.g. local_resource) shows no dropdown

Adds a structured `container` field to every log line emitted by the
pod log streaming controller, using the existing logger.Fields mechanism.
This lets the frontend filter logs by container without parsing log text.

Signed-off-by: Brent Hiranaka <brenthiranaka95@gmail.com>
@breezy-devs breezy-devs force-pushed the container-log-filter branch 3 times, most recently from 7edc4b4 to 4c8e85e Compare May 10, 2026 18:19
@breezy-devs breezy-devs changed the title add container filter to log pane log pane: filter by container for multi-container pods May 10, 2026
@breezy-devs breezy-devs force-pushed the container-log-filter branch 2 times, most recently from efc73da to f7bb2a4 Compare May 10, 2026 18:26
@breezy-devs breezy-devs changed the title log pane: filter by container for multi-container pods feat(web): add per-container log filtering to the log pane May 10, 2026
@breezy-devs breezy-devs force-pushed the container-log-filter branch 4 times, most recently from 0566c1b to 68fa027 Compare May 10, 2026 19:52
Adds a container filter dropdown to the action bar when a resource has
logs from 2 or more distinct containers. Selection is URL-persisted via
the existing search param pattern and integrates with the existing
FilterSet and log display pipeline.

Signed-off-by: Brent Hiranaka <brenthiranaka95@gmail.com>
@breezy-devs breezy-devs force-pushed the container-log-filter branch from 68fa027 to fa84275 Compare May 10, 2026 19:53
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.

Feature Request: Include/Exclude containers from logs

1 participant