Well, `if maintenance_mode?` results in a warning which is accurate, because whether this run or something else placed it into maintenance mode, if it fails, this handler will leave the node in that mode either way.  But I think I see your point - anyone familiar with how this is supposed to work (i.e. us) will not expect the handlers to take it out of maintenance mode if it didn't put it into maintenance mode, so in that sense I suppose you are saying that "leaving node" implies atypical behaviour, and it's not atypical.  So I'll change it.