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

Sanity check failed due to message output #14

Open
icoris opened this issue Sep 20, 2016 · 4 comments
Open

Sanity check failed due to message output #14

icoris opened this issue Sep 20, 2016 · 4 comments

Comments

@icoris
Copy link

icoris commented Sep 20, 2016

Hi,

I've been testing the fetcher on my program and I always get a Sanity check fail in fetch(). I've added additional debug messaging to show tokenIn and tokenOut, and this is returned:
2016/09/20 08:02:18 [overseer master] sanity check failed:d57f1402568b9d42, 2016/09/20 08:02:18 INFO:EcomRate.bdb-ConfigLoad - Retry - 5

It seems that the first message that is returned on stdOUT from my code is a database load INFO message, instead of the token. This is most likely occuring due to my import load order and my defined init.

For a quick fix I'm going to comment out the tokenIn/tokenOut check. But for a longer term fix I think Overseer needs to have a init function defined that outputs the token.

@jpillora
Copy link
Owner

It needs to be documented though, before you print anything to stdout, at the top of main run overseer.SanityCheck().

@yangyuan6
Copy link

image
image
I also encountered this problem and found the error by modifying the code debug.
So, you should remove the output before starting

@yangyuan6
Copy link

image
image

Delete output before overseer.Run(overseer.Config !

@yangyuan6
Copy link

See pull-request #72

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants