Remove useless CancelledError logging (ready for review)

We timeout polling for active status currently.  And log a nice message to that effect.  However, in order to mark the job as failed, an error gets propagated up to the controller. 

This error is a CancelledError with no other useful info - instead, propagate an error that contains the useful info rather than logging that separately.

The relevant timeout info doesn't necessarily get passed to the logger as a structured object, though.