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

Wrong Batch Transform exit code for failed jobs #97

Closed
ilazakis opened this issue Sep 16, 2019 · 1 comment
Closed

Wrong Batch Transform exit code for failed jobs #97

ilazakis opened this issue Sep 16, 2019 · 1 comment
Labels
bug Something isn't working

Comments

@ilazakis
Copy link
Contributor

ilazakis commented Sep 16, 2019

When a batch transform job fails, Sagify still returns a 0 exit code.

We need to check the SageMaker status returned here and exit with 1 or similar if the status is Failed (not sure if we can future proof this in case SageMaker changes the literal from Failed to something else but should be easy to catch).

@ilazakis ilazakis added the bug Something isn't working label Sep 16, 2019
@pm3310
Copy link
Contributor

pm3310 commented Oct 7, 2019

#98

@pm3310 pm3310 closed this as completed Oct 7, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants