Refactor: various: always use constants for OCF/executor codes

... instead of integers directly, for readability and robustness.

The only actual change is that construct_op() now initializes the exit status
to PCMK_OCF_UNKNOWN (193) rather than the nonsensical value -1, but the current
code always overwrites it before use anyway.