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

Deploy run as part of CI job fails silently #49

Closed
lennym opened this issue Jun 6, 2016 · 3 comments
Closed

Deploy run as part of CI job fails silently #49

lennym opened this issue Jun 6, 2016 · 3 comments
Labels

Comments

@lennym
Copy link
Contributor

lennym commented Jun 6, 2016

If the deployment of a lambda fails for missing Role credentials in the environment variables then the process still exits with a zero exit code and CI environments report that the deployment was successful until you view the logs to find errors.

Failed deployments should exit with a non-zero exit code.

@lennym lennym self-assigned this Jun 6, 2016
@nelsonic nelsonic added the bug label Jun 6, 2016
@tomgco
Copy link

tomgco commented Jun 6, 2016

👍 This also exhibits itself when no credentials are supplied.

@nelsonic
Copy link
Member

nelsonic commented Jun 6, 2016

@lennym good point. should it throw an error if the credentials aren't set or if deploy errors?
(how did you see this, deploying locally or from a particular Lambda?)

@lennym
Copy link
Contributor Author

lennym commented Jun 6, 2016

It was the deployment job for the new filter provider lambda in codeship.

Local deployments aren't so much of an issue since generally they'll be done by a person monitoring the logs, and not paying too much attention to the exit code.

@lennym lennym changed the title Deploy run as part of CI job exits fails silently Deploy run as part of CI job fails silently Jun 6, 2016
nelsonic added a commit that referenced this issue Jun 6, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants