Fix module.run state for non-dict return data

[terminalmage]

This backports part of a fix for this that is already in the 2014.7 branch,
properly handling execution module functions which return non-dict data.

[cachedout]

I was under the impression that there would be no more 2014.1 releases. Is that incorrect?

[terminalmage]

Yeah, I don't think we will, but I figured that in the unlikely event we do, we can at least get this one fixed.

[ghost]

Test Failed.

If the failures are unrelated to your code, don't stress, a core developer will know these apart.
In the future, if possible, please branch off a passing commit to avoid false positives.
Refer to this link for build results: http://jenkins.saltstack.com/job/salt-pr-build/9516/

[garthk]

I think I'm running into this with tls.create_self_signed_cert:

      ID: default-key
Function: module.run
    Name: tls.create_self_signed_cert
  Result: False
 Comment: An exception occurred in this state: Traceback (most recent call last):
            File "/usr/lib/python2.7/dist-packages/salt/state.py", line 1382, in call
              **cdata['kwargs'])
            File "/usr/lib/python2.7/dist-packages/salt/states/module.py", line 203, in run
              if ret['changes'].get('ret', {}).get('retcode', 0) != 0:
          AttributeError: 'str' object has no attribute 'get'

How can I track progress toward this fix being available via apt-get with the official repo? I'm on 2014.1.13+ds-1trusty1.

[cachedout]

Hi @garthk. The next release of salt, 2014.7 is currently being packaged and should be available in the repos in the next several days.