So with this PR the return value from checkjdstatus would be "Completed with Errors", and it does look like there are places that are looking for a return value of "Failed", e.g. here:

https://github.com/dell-asm/dell-idrac/blob/master/lib/puppet/provider/exporttemplatexml.rb#L47

I don't understand how the previous code would have returned "Failed" either though, as that value was only returned if the Message was "Completed with Errors", which it's not. It's "Import of system configuration XML file operation completed with errors." in this case.