Thanks for testing and the feedback.
bq: When mis-typing, such as omitting the -z, we print the error msg followed by the full usage. Suggest we instead print the helpful error message followed by Type bin/solr zk -help for usage help
Done. The long help messages do, indeed, get in the way of figuring out what was wrong.
bq: This log msg from CloudSolrClient is annoying:
Made it into a DEBUG rather than INFO.
bq: Typo: Name of the...
bq: The command bin/solr zk rm -r / succeeds, rendering Solr useless
Well don't do that . I put in a check and the script now barfs in that situation.
bq: Why do we write "Solr MUST be started on.....
because when I was copying things around during the original upconfig/downconfig I used CloudSolrClient like other tools in SolrCLI did and there is a check to see if Zookeeper has been initialized. In other words because I didn't look at it carefully enough I just proofed out using SolrZkClient and at least it works on the 'cp' command. So I should be able to convert the rest. That's annoyed me for quite a while but I never tried getting around it. Now I have.
bq: Could we wish for a solr zk ls command?
Yep, we sure could. Testing the cp command just made me wish for one, I think I'll put it in while I'm at it.
Thanks again! Probably get another patch up this weekend with all this, including the ls command.