Merge cockroachdb#103919

103919: ui: remove Circuit Breaker Tripped Events as useless r=koorosh a=koorosh Circuit Breaker Tripped Events doesn't provide extra information on top of existing `Circuit Breaker Tripped Replicas` chart. Release note (ui change): Circuit Breaker Tripped Events chart is removed from Replication dashboard. Resolves: cockroachdb#81571 Co-authored-by: Andrii Vorobiov <[email protected]>
srosenberg · May 29, 2023 · f676a1a · f676a1a
2 parents 7024705 + b481908
commit f676a1a
Showing 1 changed file with 0 additions and 18 deletions.
diff --git a/.../workspaces/db-console/src/views/cluster/containers/nodeGraphs/dashboards/replication.tsx b/.../workspaces/db-console/src/views/cluster/containers/nodeGraphs/dashboards/replication.tsx
@@ -20,7 +20,6 @@ import {
   storeIDsForNode,
 } from "./dashboardUtils";
 import {
-  CircuitBreakerTrippedEventsTooltip,
   CircuitBreakerTrippedReplicasTooltip,
   LogicalBytesGraphTooltip,
   PausedFollowersTooltip,
@@ -253,23 +252,6 @@ export default function (props: GraphDashboardProps) {
         ))}
       </Axis>
     </LineGraph>,
-    <LineGraph
-      title="Circuit Breaker Tripped Events"
-      sources={storeSources}
-      tooltip={CircuitBreakerTrippedEventsTooltip}
-    >
-      <Axis label="events">
-        {nodeIDs.map(nid => (
-          <Metric
-            key={nid}
-            name="cr.store.kv.replica_circuit_breaker.num_tripped_events"
-            title={nodeDisplayName(nodeDisplayNameByID, nid)}
-            sources={storeIDsForNode(storeIDsByNodeID, nid)}
-            nonNegativeRate
-          />
-        ))}
-      </Axis>
-    </LineGraph>,
     <LineGraph
       title="Paused Followers"
       sources={storeSources}