[Infra]Replace EuiErrorBoundary with KibanaErrorBoundary on the Infra Inventory (#226869)

jennypavlova · kibanamachine · jennypavlova · commit ff23c71d1a2f · 2025-07-08T15:36:45.000Z
Part of #225972 5 of 5 ## Summary This PR replaces `EuiErrorBoundary` with `KibanaErrorBoundary` on the metrics routing level and removes `EuiErrorBoundary` in Infra Inventory. | Before | After | |-------|-------| | ![image](https://github.com/user-attachments/assets/a656827a-c4b1-4ada-852c-bd946efe845e) | <img width="1724" alt="Screenshot 2025-07-07 at 19 06 17" src="https://github.com/user-attachments/assets/b78cb9a4-e3e9-48f1-b5dc-14292ab756e5" /> | ## Testing - Introduce an error in the metrics page (maybe a typo, non-existent component, or anything) <img width="854" alt="Screenshot 2025-07-07 at 19 07 47" src="https://github.com/user-attachments/assets/0f615c0f-60c6-4e8e-b10f-adbd93dfb502" /> - Open http://localhost:5601/ftw/app/metrics/inventory - The error should be visible, and it should still work as before (but also including telemetry) <img width="1724" alt="Screenshot 2025-07-07 at 19 06 17" src="https://github.com/user-attachments/assets/d24cf1a7-86f9-4986-a152-8a78bbe31c2e" /> --------- Co-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com> (cherry picked from commit 07e4a05)
diff --git a/x-pack/solutions/observability/plugins/infra/public/pages/metrics/inventory_view/index.tsx b/x-pack/solutions/observability/plugins/infra/public/pages/metrics/inventory_view/index.tsx
@@ -5,7 +5,6 @@
  * 2.0.
  */
 
-import { EuiErrorBoundary } from '@elastic/eui';
 import React from 'react';
 import { useTrackPageview } from '@kbn/observability-shared-plugin/public';
 import { APP_WRAPPER_CLASS } from '@kbn/core/public';
@@ -33,32 +32,30 @@ export const SnapshotPage = () => {
   ]);
 
   return (
-    <EuiErrorBoundary>
-      <WaffleOptionsProvider>
-        <WaffleTimeProvider>
-          <WaffleFiltersProvider>
-            <div className={APP_WRAPPER_CLASS}>
-              <InfraPageTemplate
-                onboardingFlow={OnboardingFlow.Infra}
-                pageHeader={{
-                  pageTitle: inventoryTitle,
-                  rightSideItems: [<SavedViews />, <SurveySection />],
-                }}
-                pageSectionProps={{
-                  contentProps: {
-                    css: css`
-                      ${fullHeightContentStyles};
-                      padding-bottom: 0;
-                    `,
-                  },
-                }}
-              >
-                <SnapshotContainer />
-              </InfraPageTemplate>
-            </div>
-          </WaffleFiltersProvider>
-        </WaffleTimeProvider>
-      </WaffleOptionsProvider>
-    </EuiErrorBoundary>
+    <WaffleOptionsProvider>
+      <WaffleTimeProvider>
+        <WaffleFiltersProvider>
+          <div className={APP_WRAPPER_CLASS}>
+            <InfraPageTemplate
+              onboardingFlow={OnboardingFlow.Infra}
+              pageHeader={{
+                pageTitle: inventoryTitle,
+                rightSideItems: [<SavedViews />, <SurveySection />],
+              }}
+              pageSectionProps={{
+                contentProps: {
+                  css: css`
+                    ${fullHeightContentStyles};
+                    padding-bottom: 0;
+                  `,
+                },
+              }}
+            >
+              <SnapshotContainer />
+            </InfraPageTemplate>
+          </div>
+        </WaffleFiltersProvider>
+      </WaffleTimeProvider>
+    </WaffleOptionsProvider>
   );
 };
