Fix proposal for #36804 #38668

wants to merge 1 commit into


None yet

3 participants


This changes it so that the batch return is returned to salt.states.saltmod.state(). I'm not entirely certain how this will impact _run_batch() in salt/cli/, however. I tried dropping in logging and launching pudb from the block of code that checks the retcode (see here), and the flow does not appear to hit that block of code. I can dig further, but I'd also like to get @cachedout's input on this fix. I don't want to fix something just to break something else inadvertently.

@terminalmage terminalmage Return the ret data from batch execution instead of raw data
Fixes #38604
@terminalmage terminalmage requested a review from cachedout Jan 10, 2017

Go Go Jenkins!


Jenkins testing

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment