We don't honor -on-error flag from packer

My understanding is that `packer build -on-error=ask` should ask me what to do.

We do have `keep_vm`, but it seems like it's ... redundant? IMO we should just let the -on-error=ask let me decide what to do.

These options are nice for debugging.