From 3e79c8f82603f182e60aa1a443c6447a9837afc7 Mon Sep 17 00:00:00 2001 From: Christopher Tubbs Date: Thu, 22 Oct 2026 19:02:58 -0400 Subject: [PATCH] Miscellaneous monitor updates * Make Activity menu to be a dropdown-menu-end type, so "Compaction Overview" doesn't extend off screen to the right * Improve accessibility * Use semantic h1 headers while keeping the sizing of h2/h3 headers * add aria- tags, as needed * Remove legacy bootstrap 3/4 items in favor of bootstrap 5.3 conventions * Make main viewport body a flex type with minimum vertical height using the entire viewable area, so the footer sticks to the bottom of the screen when the content in the main body is sparse * Fix caption on external compaction table * Fix quoting typo on recovery template * Handle case where resourceGroup is null on server template * HTML-escape tableId Co-authored-by: OpenAI GPT-6 Luna --- .../accumulo/monitor/templates/alerts.ftl | 12 +-- .../accumulo/monitor/templates/compactors.ftl | 4 +- .../monitor/templates/coordinator.ftl | 8 +- .../accumulo/monitor/templates/default.ftl | 12 +-- .../apache/accumulo/monitor/templates/ec.ftl | 101 +++++++++--------- .../accumulo/monitor/templates/fate.ftl | 4 +- .../apache/accumulo/monitor/templates/gc.ftl | 6 +- .../accumulo/monitor/templates/manager.ftl | 4 +- .../accumulo/monitor/templates/modals.ftl | 60 +++++------ .../accumulo/monitor/templates/navbar.ftl | 51 +++++---- .../accumulo/monitor/templates/overview.ftl | 8 +- .../accumulo/monitor/templates/recovery.ftl | 12 +-- .../accumulo/monitor/templates/scans.ftl | 4 +- .../accumulo/monitor/templates/server.ftl | 6 +- .../accumulo/monitor/templates/sservers.ftl | 4 +- .../accumulo/monitor/templates/table.ftl | 18 ++-- .../monitor/templates/table_loading.ftl | 2 +- .../accumulo/monitor/templates/tables.ftl | 14 ++- .../accumulo/monitor/templates/tservers.ftl | 4 +- 19 files changed, 166 insertions(+), 168 deletions(-) diff --git a/server/monitor/src/main/resources/org/apache/accumulo/monitor/templates/alerts.ftl b/server/monitor/src/main/resources/org/apache/accumulo/monitor/templates/alerts.ftl index 7422fff65a2..2eef62473d3 100644 --- a/server/monitor/src/main/resources/org/apache/accumulo/monitor/templates/alerts.ftl +++ b/server/monitor/src/main/resources/org/apache/accumulo/monitor/templates/alerts.ftl @@ -19,15 +19,15 @@ -->
-
-
Alerts
+
+

Alerts

+ class="table table-bordered table-striped table-sm" style="width: 100%;"> - - - + + + diff --git a/server/monitor/src/main/resources/org/apache/accumulo/monitor/templates/compactors.ftl b/server/monitor/src/main/resources/org/apache/accumulo/monitor/templates/compactors.ftl index bed7793e2ef..6e0509e7586 100644 --- a/server/monitor/src/main/resources/org/apache/accumulo/monitor/templates/compactors.ftl +++ b/server/monitor/src/main/resources/org/apache/accumulo/monitor/templates/compactors.ftl @@ -22,13 +22,13 @@
-
+
Compactors
The following Compactors reported status.

-
PriorityCategoryMessagePriorityCategoryMessage
+
<#include "table_loading.ftl" >
diff --git a/server/monitor/src/main/resources/org/apache/accumulo/monitor/templates/coordinator.ftl b/server/monitor/src/main/resources/org/apache/accumulo/monitor/templates/coordinator.ftl index 70564e5d117..a420001f969 100644 --- a/server/monitor/src/main/resources/org/apache/accumulo/monitor/templates/coordinator.ftl +++ b/server/monitor/src/main/resources/org/apache/accumulo/monitor/templates/coordinator.ftl @@ -21,19 +21,19 @@
Compaction Coordinator Activity
- +
<#include "table_loading.ftl" >

Compaction Coordinator Queues
- +
<#include "table_loading.ftl" >

Running Compactions By Table
- +
@@ -45,7 +45,7 @@
Running Compactions By Queue
-
Table Name
+
diff --git a/server/monitor/src/main/resources/org/apache/accumulo/monitor/templates/default.ftl b/server/monitor/src/main/resources/org/apache/accumulo/monitor/templates/default.ftl index 9b373ce14c0..eac06f4e111 100644 --- a/server/monitor/src/main/resources/org/apache/accumulo/monitor/templates/default.ftl +++ b/server/monitor/src/main/resources/org/apache/accumulo/monitor/templates/default.ftl @@ -19,11 +19,12 @@ --> - +${title} - Accumulo ${version} - + + - + <#include "navbar.ftl"> @@ -89,10 +90,9 @@ View alerts -
+
<#include "${template}"> -
-
+
diff --git a/server/monitor/src/main/resources/org/apache/accumulo/monitor/templates/ec.ftl b/server/monitor/src/main/resources/org/apache/accumulo/monitor/templates/ec.ftl index 5fb41c8adff..584611e4bae 100644 --- a/server/monitor/src/main/resources/org/apache/accumulo/monitor/templates/ec.ftl +++ b/server/monitor/src/main/resources/org/apache/accumulo/monitor/templates/ec.ftl @@ -19,64 +19,63 @@ -->
-
-

${title}

+
+

${title}

-
-
Queue Name
- + + +
-
-
-

- -

-
-
- -
- - - -
- -
- - - -
- -
- - - -
- -
- - - -
- -
- - - - Valid formats: >10m, <1h, >=5s, <=2d -
- -
- -
-
+
+
+
+

+ +

+
+
+ +
+ + + +
+ +
+ + + +
+ +
+ + + +
+ +
+ + + +
+ +
+ + + + Valid formats: >10m, <1h, >=5s, <=2d +
+ +
+
-
+ diff --git a/server/monitor/src/main/resources/org/apache/accumulo/monitor/templates/fate.ftl b/server/monitor/src/main/resources/org/apache/accumulo/monitor/templates/fate.ftl index 3041a0fbbc5..7beb5500823 100644 --- a/server/monitor/src/main/resources/org/apache/accumulo/monitor/templates/fate.ftl +++ b/server/monitor/src/main/resources/org/apache/accumulo/monitor/templates/fate.ftl @@ -19,8 +19,8 @@ -->
-
-
Running External Compactions
Server Hostname
+
+
diff --git a/server/monitor/src/main/resources/org/apache/accumulo/monitor/templates/gc.ftl b/server/monitor/src/main/resources/org/apache/accumulo/monitor/templates/gc.ftl index 93bb069fa65..aaf4080402d 100644 --- a/server/monitor/src/main/resources/org/apache/accumulo/monitor/templates/gc.ftl +++ b/server/monitor/src/main/resources/org/apache/accumulo/monitor/templates/gc.ftl @@ -24,21 +24,21 @@
Garbage Collector
-
Fate Transaction Details
Use the accumulo inst fate server-side utility with the -l and -i options to see detailed information about a transaction.
+
<#include "table_loading.ftl" >
File Collection
- +
<#include "table_loading.ftl" >
Wal Collection
- +
<#include "table_loading.ftl" >
diff --git a/server/monitor/src/main/resources/org/apache/accumulo/monitor/templates/manager.ftl b/server/monitor/src/main/resources/org/apache/accumulo/monitor/templates/manager.ftl index f011fd46fd5..2fea40ee6a9 100644 --- a/server/monitor/src/main/resources/org/apache/accumulo/monitor/templates/manager.ftl +++ b/server/monitor/src/main/resources/org/apache/accumulo/monitor/templates/manager.ftl @@ -29,13 +29,13 @@ Managers
- +
<#include "table_loading.ftl" >

Manager Fate Activity
- +
<#include "table_loading.ftl" >

diff --git a/server/monitor/src/main/resources/org/apache/accumulo/monitor/templates/modals.ftl b/server/monitor/src/main/resources/org/apache/accumulo/monitor/templates/modals.ftl index 8d4aeed7a5a..af0d467df80 100644 --- a/server/monitor/src/main/resources/org/apache/accumulo/monitor/templates/modals.ftl +++ b/server/monitor/src/main/resources/org/apache/accumulo/monitor/templates/modals.ftl @@ -18,50 +18,46 @@ under the License. --> - -