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