Good spot, except in general this still should be the hostname of the sprout cluster - it's only the public_hostname when we use clearwater-auto-config (i.e. on all-in-one nodes).  It might be best to leave as-is.

Having said that, it's wrong for it to currently say "load-balancer", as we got rid of these a long time ago.  Are you OK to update the descriptions for sprout_hostname, hs_hostname, xdms_hostname to say "cluster" instead?