The executor events weren't being stored
Changed approach to log from the step_delegating_executor rather than the step handlers, because the executor has the step contexts and can log correctly. this also means you don't have to remember to log the individual events.
concern with this approach is accidentally getting dupe events if anyone returns an event that was already logged (ie it was created with one of the static constructors that logs)
maybe just add a test like ** that checks for dupes?
tangential: should this not have some sort of message to indicate that the failure is due to the k8s job failing?