Add kwargs to returner data.

[n-holmstedt]

Is your feature request related to a problem? Please describe.
Using a returner for event based automation. I'd like to see the key-value arguments passed to the salt-function.

Describe the solution you'd like
Could one append kwargs to fun_args in the data sent to the returner? I believe napalm does this.

Describe alternatives you've considered
forking.

Additional context
none

[mirceaulinic]

I think that should be supported:

  --return=RETURNER     The name of the Returner module to use for sending
                        data to various external systems.
  --return-config=RETURNER_CONF
                        Specify an alternative Returner config.
  --return-kwargs=RETURNER_KWARGS
                        Set Returner options at the command line.

Or is there something different you'd like? Please explain.

[n-holmstedt]

Sorry, bad explanation,

The command
salt-sproxy minion module.function foo bar=baz --return=my_module
yields the following data structure to the return module:

{
  'id': 'minion',
  'fun': 'module.function',
  'fun_args: ['foo']
  ...
}

I would like it to do:

{
  'id': 'minion',
  'fun': 'module.function',
  'fun_args: ['foo' {'bar':'baz'}]
  ...
}

I can make it do exactly that by appending the kwargs to the args list in salt_call before ret_data is populated

    args.append(kwargs)
    ret_data = {
                'id': opts['id'],
                'jid': jid,

Is that a bit clearer? Bit hard to explain i guess.

[mirceaulinic]

Ah, yes, I see it now. Yep, @n-holmstedt - you solution sound good, please feel free to open a PR if you would like to! 😄