web: add container filter to log pane

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.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Signed-off-by: Brent Hiranaka <brenthiranaka95@gmail.com>

Author: breezy-devs

web/src/CopyLogs.test.tsx

@@ -23,6 +23,7 @@ const DEFAULT_FILTER_SET: FilterSet = {
   source: FilterSource.all,
   level: FilterLevel.all,
   term: EMPTY_FILTER_TERM,
+  containers: [],
 }
 
 describe("CopyLogs", () => {

web/src/LogStore.ts

@@ -418,6 +418,24 @@ class LogStore implements LogAlertIndex {
     return result
   }
 
+  // Returns the distinct container names seen in log lines for the given manifest.
+  // Only includes names from lines that carry a container field (pod logs).
+  // Returns an empty array for resources with no multi-container log data.
+  containersForManifest(mn: string): string[] {
+    const manifestSpans = this.spansForManifest(mn)
+    const seen = new Set<string>()
+    for (const line of this.lines) {
+      if (!manifestSpans[line.spanId]) {
+        continue
+      }
+      const name = line.fields?.container
+      if (name) {
+        seen.add(name)
+      }
+    }
+    return Array.from(seen)
+  }
+
   getOrderedBuildSpanIds(spanId: string): string[] {
     let startSpan = this.spans[spanId]
     if (!startSpan) {
@@ -630,6 +648,7 @@ class LogStore implements LogAlertIndex {
           level: storedLine.level,
           manifestName: span.manifestName,
           buildEvent: storedLine.fields?.buildEvent,
+          containerName: storedLine.fields?.container,
           spanId: spanId,
           storedLineIndex: i,
         }

web/src/OverviewActionBar.test.tsx

@@ -11,18 +11,24 @@ import { useLocation } from "react-router-dom"
 import { SnackbarProvider } from "notistack"
 import React from "react"
 import { ButtonSet } from "./ApiButton"
-import { EMPTY_FILTER_TERM, FilterLevel, FilterSource } from "./logfilters"
+import {
+  EMPTY_FILTER_TERM,
+  FilterLevel,
+  FilterSet,
+  FilterSource,
+} from "./logfilters"
 import OverviewActionBar, {
   createLogSearch,
   FILTER_INPUT_DEBOUNCE,
 } from "./OverviewActionBar"
 import { EmptyBar, FullBar } from "./OverviewActionBar.stories"
 import { disableButton, oneResource, oneUIButton } from "./testdata"
 
-const DEFAULT_FILTER_SET = {
+const DEFAULT_FILTER_SET: FilterSet = {
   level: FilterLevel.all,
   source: FilterSource.all,
   term: EMPTY_FILTER_TERM,
+  containers: [],
 }
 
 let location: any = window.location
@@ -257,6 +263,104 @@ describe("OverviewActionBar", () => {
     })
   })
 
+  describe("FilterContainerMenu", () => {
+    const containers = ["app", "istio-proxy", "daprd"]
+
+    it("does not render when fewer than 2 containers are present", () => {
+      customRender(
+        <OverviewActionBar
+          filterSet={DEFAULT_FILTER_SET}
+          logContainers={["app"]}
+        />
+      )
+      expect(
+        screen.queryByLabelText(/select containers to filter logs/i)
+      ).toBeNull()
+    })
+
+    it("renders when 2 or more containers are present", () => {
+      customRender(
+        <OverviewActionBar
+          filterSet={DEFAULT_FILTER_SET}
+          logContainers={["app", "istio-proxy"]}
+        />
+      )
+      expect(
+        screen.getByLabelText(/select containers to filter logs/i)
+      ).toBeInTheDocument()
+    })
+
+    it("shows the container name in the right pill when 1 container is selected", () => {
+      customRender(
+        <OverviewActionBar
+          filterSet={{ ...DEFAULT_FILTER_SET, containers: ["app"] }}
+          logContainers={containers}
+        />
+      )
+      expect(
+        screen.getByLabelText(/select containers to filter logs/i)
+      ).toHaveTextContent("app")
+    })
+
+    it("shows a count in the right pill when 2+ containers are selected", () => {
+      customRender(
+        <OverviewActionBar
+          filterSet={{
+            ...DEFAULT_FILTER_SET,
+            containers: ["app", "istio-proxy"],
+          }}
+          logContainers={containers}
+        />
+      )
+      expect(
+        screen.getByLabelText(/select containers to filter logs/i)
+      ).toHaveTextContent("2")
+    })
+
+    it("updates the URL when a container is toggled on", () => {
+      customRender(
+        <OverviewActionBar
+          filterSet={DEFAULT_FILTER_SET}
+          logContainers={containers}
+        />
+      )
+      userEvent.click(
+        screen.getByLabelText(/select containers to filter logs/i)
+      )
+      userEvent.click(screen.getByRole("menuitemcheckbox", { name: /app/i }))
+      expect(getSearch()).toContain("containers=")
+      expect(new URLSearchParams(getSearch()).get("containers")).toBe("app")
+    })
+
+    it("removes a container from the URL when it is toggled off", () => {
+      customRender(
+        <OverviewActionBar
+          filterSet={{ ...DEFAULT_FILTER_SET, containers: ["app"] }}
+          logContainers={containers}
+        />
+      )
+      userEvent.click(
+        screen.getByLabelText(/select containers to filter logs/i)
+      )
+      userEvent.click(screen.getByRole("menuitemcheckbox", { name: /app/i }))
+      expect(new URLSearchParams(getSearch()).get("containers")).toBeNull()
+    })
+
+    it("clears the containers filter when 'All Containers' is clicked", () => {
+      customRender(
+        <OverviewActionBar
+          filterSet={{ ...DEFAULT_FILTER_SET, containers: ["app"] }}
+          logContainers={containers}
+        />
+      )
+      userEvent.click(
+        screen.getByLabelText(/select containers to filter logs/i)
+      )
+      userEvent.click(screen.getByRole("menuitem", { name: /all containers/i }))
+      expect(new URLSearchParams(getSearch()).get("containers")).toBeNull()
+    })
+  })
+
   describe("createLogSearch", () => {
     let currentSearch: URLSearchParams
     beforeEach(() => (currentSearch = new URLSearchParams()))
@@ -311,5 +415,31 @@ describe("OverviewActionBar", () => {
         }).toString()
       ).toBe("source=build&term=test")
     })
+
+    it("sets the containers param when a non-empty array is passed", () => {
+      expect(
+        createLogSearch(currentSearch.toString(), {
+          containers: ["app", "istio"],
+        }).get("containers")
+      ).toBe("app,istio")
+    })
+
+    it("removes the containers param when an empty array is passed", () => {
+      currentSearch.set("containers", "app,istio")
+      expect(
+        createLogSearch(currentSearch.toString(), { containers: [] }).get(
+          "containers"
+        )
+      ).toBeNull()
+    })
+
+    it("preserves an existing containers param when containers is undefined", () => {
+      currentSearch.set("containers", "app")
+      expect(
+        createLogSearch(currentSearch.toString(), { term: "test" }).get(
+          "containers"
+        )
+      ).toBe("app")
+    })
   })
 })