Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Synchronous module calls via salt-ssh #161

Merged
merged 15 commits into from Jul 4, 2016

Conversation

@renner
Copy link
Member

commented Jun 22, 2016

This patch adds support for calling LocalCall instances synchronously via salt-ssh. The results of calls to state.apply were further refined to be parsed into the new type State.ApplyResult that contains the changes unparsed as JSONElement.

@renner renner force-pushed the master-salt-ssh branch from 8c4cf30 to 32a4f67 Jun 27, 2016

renner added 11 commits Jun 20, 2016
The 'return' attribute can be null
This seems to be the case for hosts that are defined in the roster but don't actually exist.

@renner renner force-pushed the master-salt-ssh branch from 9062dc3 to 567dd03 Jun 28, 2016

renner added 2 commits Jun 28, 2016

@renner renner changed the title [WIP] Add support for synchronous module calls via salt-ssh Add support for synchronous module calls via salt-ssh Jul 1, 2016

renner added 2 commits Jul 3, 2016

@renner renner changed the title Add support for synchronous module calls via salt-ssh Synchronous module calls via salt-ssh Jul 3, 2016

@renner renner added this to the Version 0.8.0 milestone Jul 3, 2016

@renner

This comment has been minimized.

Copy link
Member Author

commented Jul 4, 2016

@lucidd: Could you please have another look at the patch before I merge it?

@renner renner merged commit 4a6e1de into master Jul 4, 2016

2 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details

@renner renner deleted the master-salt-ssh branch Jul 4, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
1 participant
You can’t perform that action at this time.