Rudder

Attached the failure's log

I see what is happening here.

The test is defined as:

vars:
  "kept_codes" string => "0,1, 2";
  "repaired_codes" string => "3, 4";

methods:
  "phE" usebundle => command_execution_result("exit 5", "${kept_codes}", "${repaired_codes}");

But the command_execution_result method uses a classes result body defined as follows:

body classes classes_generic_return_code_list(x, kept_return_codes, repaired_return_codes)                                                                                                                                                    
{
  kept_returncodes => { @{kept_return_codes} };
  repaired_returncodes => { @{repaired_return_codes} };

  promise_repaired => { "promise_repaired_$(x)", "$(x)_repaired", "$(x)_ok", "$(x)_reached" };
  repair_failed => { "repair_failed_$(x)", "$(x)_failed", "$(x)_not_ok", "$(x)_not_kept", "$(x)_not_repaired", "$(x)_reached", "${x}_error" };
  repair_denied => { "repair_denied_$(x)", "$(x)_denied", "$(x)_not_ok", "$(x)_not_kept", "$(x)_not_repaired", "$(x)_reached", "${x}_error"  };  
  repair_timeout => { "repair_timeout_$(x)", "$(x)_timeout", "$(x)_not_ok", "$(x)_not_kept", "$(x)_not_repaired", "$(x)_reached", "${x}_error"  };  
  promise_kept => { "promise_kept_$(x)", "$(x)_kept", "$(x)_ok", "$(x)_not_repaired", "$(x)_reached" };
}

As you can see, "error_returncodes" is not defined.

However, the CFEngine documentation is quite clear about what will happen here (from https://docs.cfengine.com/lts/reference-promise-types.html#failed_returncodes)

If none of the attributes kept_returncodes, repaired_returncodes, or failed_returncodes are set, the default is to consider a return code zero as promise repaired, and nonzero as promise failed.

That means none of them, as in zero. However, we define two, so the default does not come into play. This means that any return codes other than the success and repaired codes specified will not cause result classes to be defined.

We either need to change the default behaviour in CFEngine or find a way to say "failed_returncodes = <others>".

This was fixed in CFEngine 3.6.1 (CFEngine Redmine #5986), so any more recent version will work.

Especially, all currently supported Rudder releases uses a higher version.

Closing.