HomeClusterLabs Projects

Low: daemons: Send a NACK instead of an ACK when there's an error.
821786b7f53bUnpublished

Unpublished Commit ยท Learn More

Not On Permanent Ref: This commit is not an ancestor of any permanent ref.

Description

Low: daemons: Send a NACK instead of an ACK when there's an error.

I don't know the reasoning behind this, but it seems strange for daemons
to send an ACK along with an error code. For starters, it seems a
little redundant to have both an ACK vs. NACK, as well as a status code.

But since we have that, it's confusing to send both an ACK (which seems
like a positive response) and an error code (which is obviously not).

So, anywhere we'd respond with an error code (aside from
CRM_EX_INDETERMINATE which means we don't know yet), send a NACK
instead.

Details

Provenance
clumensAuthored on Mon, Jul 21, 2:59 PM
Parents
rP34d76196bc8d: Low: daemons: Return CRM_EX_PROTOCOL when ACKing an unknown request.
Branches
Unknown
Tags
Unknown

Event Timeline