fix #327 remove unexpect error entry in deployment log

We have improved to add error detail when failed.  However, we didn't mark the final state for the deployment status (from pending/building to failed state).  This leaves inactive deployment in non-final state.  This is unexpected.

When deployments Http GET, any deployment in such state will be marked failed automatically + one entry indicating unexpected failure.

The fix is to set Failed status (MarkFailed) in the timely fashion.