chihsuan commented on code in PR #11243:
URL: https://github.com/apache/ozone/pull/11243#discussion_r4145400888
##########
ozone-ui/packages/om/src/App.tsx:
##########
@@ -89,12 +90,19 @@ function AppShell() {
<Routes>
<Route path="/" element={<OverviewPage />} />
<Route path="/configuration" element={<Placeholder
title="Configuration" />} />
- <Route path="/rpc" element={<Placeholder title="Remote Procedure Call"
/>} />
- <Route path="/ozone-manager" element={<Placeholder title="Ozone
Manager" />} />
- <Route path="/jmx-info" element={<Placeholder title="JMX" />} />
+ {/* Metrics group */}
+ <Route path="/metrics/rpc" element={<Placeholder title="Remote
Procedure Call" />} />
+ <Route
+ path="/metrics/ratis-event-timeline"
+ element={<Placeholder title="Ratis Event Timeline" />}
+ />
+ <Route path="/metrics/ozone-manager" element={<MetricsPage />} />
+ <Route path="/metrics/deletion" element={<Placeholder title="Deletion"
/>} />
+ <Route path="/metrics/snapshots" element={<Placeholder
title="Snapshots" />} />
+ {/* Common tools group */}
+ <Route path="/jmx" element={<Placeholder title="JMX" />} />
Review Comment:
Just curious, why did this change from `/jmx-info` to `/jmx`? If I reload
this page in dev, I get OM's JMX JSON instead of the app.
https://github.com/user-attachments/assets/ac4ecd8a-9674-40f4-8a52-1a6324527665
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
