The facts plan should always add facts for successful nodes

[glangloi]

Use Case

I would expect the facts plan to always add facts to the inventory for successful nodes, even if it failed on some nodes.

Describe the Solution You Would Like

The facts task should catch errors and then call add_facts for successful nodes. Then either fail the plan with the original error, or return $result_set if all succeeded. However, I could be missing an important reason for why add_facts and the facts task are independent of each other in the first place.

Describe Alternatives You've Considered

I'm currently using a wrapper plan which does the above. This lets me run facts for a bunch of nodes and at least continue on even if we failed to collect facts for some nodes.