Might be cleaner to make the relevant functions (those which set passwords that should be echoed: `configure_datastore_add_users`, `configure_datastore`, `configure_activemq`, `configure_mcollective_for_activemq_on_broker`, `configure_mcollective_for_activemq_on_node`, `configure_routing_plugin`, and `configure_httpd_auth`) assign to a global dictionary (to avoid duplicates) and then echo the values at the end.

Also, it might be better organisation to put the `echo` statements and surrounding logic in a function.