lnst.Slave.Job.py: missing json keyword when passing argument to exec_cmd()

[jtluka]

There's a bug at following line:

lnst/lnst/Slave/Job.py

Line 242 in 86c1ddd

stdout, stderr = exec_cmd(self._what["command"], self._what["json"])

The self._what["json"] value is passed as value of die_on_err argument, which makes every command running through this method return with sucess. Instead the value should be passed with the keyword, that is exec_cmd(...json=self._what["json"])

[olichtne]

Good catch, I guess this has been impacting us from the start that all host.run("<shell command>") calls always passed?

[jtluka]

Sent a patch for review, link

[jtluka]

Fixed by commit 7448449