The 'reason' key is not required by the schema, and we found some
documents at CHX that do not have it.
It's a separate question how we are ending up with documents that have
an 'exit_status' other than 'success' by no 'reason'. The RunEngine
should be ensuring that. Nonetheless, this suitcase should be tolerant
of any document that meets the schema, so this change is correct.
Yeah, this was actually just an incidental issue that we happened across while fixing the main issue at CHX. An email with DAMA CC-ed is on its way with the full story.
The 'reason' key is not required by the schema, and we found some documents at CHX that do not have it.
It's a separate question how we are ending up with documents that have an 'exit_status' other than 'success' by no 'reason'. The RunEngine should be ensuring that. Nonetheless, this suitcase should be tolerant of any document that meets the schema, so this change is correct.