I think this is us trying to be too accommodating, and too clever. I'm surprised it works in the current code. We do `if (!truth(waitForSshable))`, but if the value is the string "false" then I'd expect that to give true.

Though it might feel nice to use sometimes (and look nice in a demo that's already written), this level of flexibility allows someone to shoot them self in the foot easily (e.g. they enter "5minuts" and it ignores the error entirely, without warning, and uses the default).