Refactor: tools: Rename a function to clean_up_on_connection_failure.

And only call it if rc is not pcmk_rc_ok. After working with this
function a little bit, the previous behavior just seemed weird. In
particular, the block in one_shot where it's called and then clean_up is
called right after is odd.