-
Notifications
You must be signed in to change notification settings - Fork 227
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
Some errors do not result in a non-zero exit code #304
Comments
igungor
pushed a commit
that referenced
this issue
Aug 5, 2021
…de error (#328) Creating two error objects instead of one, is to avoid data race. Since there is a goroutine running which tries to write merror object, there might be a data race for merror object in cp, rm and select commands. We can create 1 error object for the goroutine and 1 for the main routine, and append them at the end of the execution. Tested #304 's first case, it returns exit code 1. Resolves #304
@weisdd Hi. Could you please try with the master branch to see if the problems you experience are fixed? |
Hi @weisdd we had a PR which solves this issue but since it will break some code, we will release it in version 2.0.0, fyi. |
igungor
pushed a commit
that referenced
this issue
Sep 24, 2021
Co-authored-by: İbrahim Güngör <[email protected]> Updates #304
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
System info
OS: Alpine Linux 3.13.1
s5cmd version: v1.2.1-3fc01af
Observations
The host is resolvable and reachable, but credentials are not passed through envs:
The host was resolvable, unreachable:
The host is unresolvable:
All of that means that we can't be sure that pipelines fail in case there's something wrong with
s3cmd cp
.The text was updated successfully, but these errors were encountered: