[r=andreserl][bug=1604962,1604987][author=andreserl] Fixes to correctly log cloud-init/curtin FAIL events in the node event log. 
  
Fixes to correctly log "mark_failed" events. We did this by combining the mark_failed with _mark_failed functions, provided that the latter wasn't really storing anything in the event log and it should. It now differentiates between a user driven "Mark Failed" vs a system driven.

Drive-by fix to improve mark_failed function.