saltstack/salt — pull request #15220 by toddnni (2014-08-24 07:11:49+00:00)
Title: Fix: Make module.run state to display False mret also

Before the change, this
```
mine.send:
module.run:
- func: network.ip_add
```
returns
```
hostname:
----------
ID: mine.send
Function: module.run
Result: True
Comment: Module function mine.send executed
Changes:
```

To me it is a bug that failed execution is not visible in any way. We have two options
1. either make state to fail or
2. return False in `Changes:` in the same way as True is returned in a succesfull case.

I implemented the option 2. because in some sense module execution succeeds, but returned value is False.
After the change state returns
```
hostname:
----------
ID: mine.send
Function: module.run
Result: True
Comment: Module function mine.send executed
Changes:
----------
ret:
False
```

I'm okay with the option 1. also.