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

Rundeck timing out even if job is finished #14

Closed
tehsu opened this issue Mar 8, 2016 · 7 comments
Closed

Rundeck timing out even if job is finished #14

tehsu opened this issue Mar 8, 2016 · 7 comments

Comments

@tehsu
Copy link

tehsu commented Mar 8, 2016

I'm trying to figure out what to add to fix this issue to the returners, I figured the default ones would cover this but doesn't look like it, I added state.sls to the configuration and recompiled the jar.

[salt.state ][INFO ] {'pid': 7435, 'retcode': 0, 'stderr': '', 'stdout': '.'}

Would this parser work here?

defaultCommandParser: &defaultCommandParser !!org.rundeck.plugin.salt.output.SaltJsonReturnHandler {exitCodeKey: retcode, standardOutputKey: stdout, standardErrorKey: stderr}

@petrudumuta
Copy link

Hey @tehsu ! Did you manage to get it working when using the state.sls function?

@tehsu
Copy link
Author

tehsu commented Mar 24, 2016

I did, I had to use the fork from here, https://github.com/apenvern/salt-step

I compiled it with some changes to the defaultReturners.yml, but it works perfectly now. I can provide the changes that I made.

@petrudumuta
Copy link

Cool! Yes, if you could provide them that would be great. Thanks!

@tehsu
Copy link
Author

tehsu commented Mar 24, 2016

All you need to change is the defaultReturners.yaml, it has state.highstate: *alwaysSuccessful, copy it and change it to state.sls

@petrudumuta
Copy link

yeah, just tested and it works, thought it was something more involved :) thanks for your quick reply! almost lost a day on this :(

@tehsu
Copy link
Author

tehsu commented Mar 24, 2016

better than me, I lost a week

@tehsu
Copy link
Author

tehsu commented May 5, 2019

This needs to be merged, #19. It's the correct fix. Closing my ticket.

@tehsu tehsu closed this as completed May 5, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants