keith-turner commented on code in PR #5439:
URL: https://github.com/apache/accumulo/pull/5439#discussion_r2025393190


##########
assemble/bin/accumulo-cluster:
##########
@@ -652,6 +653,50 @@ function control_services() {
 
 }
 
+function prune(){
+  trap 'rm -f "$SERVICE_JSON"' EXIT
+  SERVICE_JSON=$(mktemp --tmpdir "accumulo-services-XXXXXXXX.json") || exit 1
+  "$accumulo_cmd" admin serviceStatus --json > $SERVICE_JSON 2>/dev/null 

Review Comment:
   Its working well so far. Manually tested the following cases of extra 
servers w/ the shell code and it find the extra stuff.
   
    * Group does not exists in cluster.yaml
    * Host does not exists in cluster.yaml
    * Count is lower than in cluster.yaml
   
   Trying to figure out the best way to generalize the code for all server type 
as next step then will look into making it call zoo zap.



-- 
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-unsubscr...@accumulo.apache.org

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

Reply via email to