-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Return code convention mismatch with Unix #1153
Comments
p.s. I'm not sure why configuration |
I am going to close this issue because the expected exit codes can already be changed with |
I am not sure. I suspect it was because some commands return exit status 2 for incorrect arguments. I think the original idea may have been that running a command with incorrect arguments again could not succeed, so don't automatically restart it by default. We are coming up on another major version of Supervisor so we have the opportunity to change the default value for |
(cherry picked from commit 837c159)
fyi, there can be issues with Go processes not restarting on failure due to
panic()
which exits the program with exit code 2.Discussion here: https://groups.google.com/forum/#!msg/golang-nuts/u9NgKibJsKI/XxCdDihFDAAJ
Go-lang issue: golang/go#24284
Thanks!
The text was updated successfully, but these errors were encountered: