[GitHub] [accumulo] DomGarguilo commented on issue #3373: Monitor no longer shows goal state

["DomGarguilo (via GitHub)" <gi...@apache.org>]

DomGarguilo commented on issue #3373:
URL: https://github.com/apache/accumulo/issues/3373#issuecomment-1536710881

   Looking at the Monitor code in 1.10 I see the following:
   
   https://github.com/apache/accumulo/blob/a71fa294f13b2176a4882b0cfb15336a5be0dcd4/server/monitor/src/main/java/org/apache/accumulo/monitor/servlets/MasterServlet.java#L123-L126
   
   Which looks like it will display on the Master page if the `state` from the monitors `MasterMonitorInfo` is other than `MasterState.NORMAL`.
   
   https://github.com/apache/accumulo/blob/a71fa294f13b2176a4882b0cfb15336a5be0dcd4/server/monitor/src/main/java/org/apache/accumulo/monitor/servlets/XMLServlet.java#L103
   
   And this is appended to the output of the XML endpoint (which i was able to verify by standing up a 1.10 cluster and viewing)
   
   
   In 2.1, it looks like that data (now `managerGoalState`) is still present in the `MangerMonitorInfo` but is never displayed on the Manger page. I think it will still be present in the `/XML` endpoint though (but have not yet verified).
   
   @ivakegg does this sound right to you? If so, is the goal of this ticket to have the manager goal state displayed in the 2.1 manger page if it is not `NORMAL`?


-- 
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: notifications-unsubscribe@accumulo.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org