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

Remove error in logs when image not found #182

Closed
treeder opened this issue Oct 18, 2016 · 7 comments
Closed

Remove error in logs when image not found #182

treeder opened this issue Oct 18, 2016 · 7 comments
Assignees
Labels
Milestone

Comments

@treeder
Copy link
Contributor

treeder commented Oct 18, 2016

This is in runner I believe.

screen shot 2016-10-18 at 4 07 24 pm

Change it to info, and only error if the image can't be pulled.

@treeder treeder added this to the Alpha 1 milestone Oct 18, 2016
@seiflotfy seiflotfy self-assigned this Oct 18, 2016
@ucirello ucirello self-assigned this Oct 18, 2016
@seiflotfy seiflotfy removed their assignment Oct 18, 2016
@ucirello
Copy link
Contributor

@treeder do you have a step by step to reproduce this error? I have tried here in many ways, but I get a different error.

@ucirello
Copy link
Contributor

I grep'd the code for "Handler for", and it turns out it is a Docker daemon error message. It is either here or here. Either case it is Docker printing, not IronFunctions. Were you running it in DinD? If so, that could be reason.

@seiflotfy
Copy link
Contributor

I can confirm that this is a DinD issue

@treeder
Copy link
Contributor Author

treeder commented Oct 19, 2016

Still should try to fix this, the default way this runs is with dind.

@treeder treeder reopened this Oct 19, 2016
@treeder
Copy link
Contributor Author

treeder commented Oct 19, 2016

Not a critical thing though, don't spent too much time on it right now, can deal with it later.

@treeder treeder removed this from the Alpha 1 milestone Oct 19, 2016
@treeder treeder removed the invalid label Oct 19, 2016
@treeder
Copy link
Contributor Author

treeder commented Nov 15, 2016

I believe this is fixed now? I don't think it shows up anymore.

@treeder treeder added this to the Alpha 2 milestone Nov 15, 2016
@ucirello
Copy link
Contributor

I think so.

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

4 participants