-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
make check error #1098
Comments
Here is all logs of 'make check' :
|
Can you provide the information contained in the issue template? Thanks! |
I provided information for each questions on issue template already. if it is not, what information do you want more ...? |
Hello, Roasbeef. Previously, I did a job on master branch. but also after changing to 0.4.1-beta released version, error occurred again. I updated information and log for my issue post. |
@xiaofo09 Do you get this error every time you run |
Thank you for suggesting solution..! But same error occurred even I tried it 3 times.
|
Alright, let's try to figure this one out :)
Thanks! |
|
I have this same error every time I run |
@xiaofo09 Sorry, should be |
@molxyz |
@halseth where do I have to uncomment those?
|
@cmdrtee Oops, was supposed to say comment out/delete the non-failing test 😛 In any case, it will be like this: aeac2ce for the case where |
@halseth I tried to do it newly, so I reinstalled OS (Ubuntu 16.04) and did what you asked. Every pre-condition is same as above. (Including bitcoind version, btcd version and all dependencies) I posted every log on Pastebin. But unlike previous situation, error occurred on TestLightningNetworkDaemon not async_payments_benchmark. Log for installing 'lnd' on 0.4.1-beta realeased : https://pastebin.com/Ev8xzF5i |
@xiaofo09 You have a compile error during EDIT: you must run |
@cmdrtee goimports is a tool for updating your Go import lines, adding missing ones and removing unreferenced ones. from https://godoc.org/golang.org/x/tools/cmd/goimports
|
I commented only below part from lnd_test.go ( after testing 'make itest' with commenting only "graph topology notifications", error occurred again about "async payments benchmark". so I commented both of them )
Here is full log : Here is the logs of output |
@xiaofo09 That failure is a known flake, I think I have a fix for it in one of my PRs. We should instead try to figure out why you are getting the (remember to |
Thank you, halseth. :) output0.log : http://pastefile.com/wVxPDD |
Thank you for the explanation and also sorry for hijacking you're issue @xiaofo09 I also went through every step as described by @halseth since I think I'm having the exact same issue. stdout.txt : https://pastebin.com/eCGbTVZ8 |
@cmdrtee The test actually fails to start. Make sure |
@halseth I actually did
|
@cmdrtee Are you still getting that failure, even after killing the processes? |
Closing due to inactivity. |
Background
When I do "make check", I have the following error :
Your environment
lnd
: lnd version 0.4.1-beta commit=6fa93a78c18180a3bc77116b749b3c5e279d5a33uname -a
on *Nix) : Linux xiaofo 4.13.0-38-generic Make routing options configurable from command line #43~16.04.1-Ubuntu SMP Wed Mar 14 17:48:43 UTC 2018 x86_64 x86_64 x86_64 GNU/Linuxbtcd
,bitcoind
, or other backend : bitcoind v0.16.0, btcd v0.12.0-betaSteps to reproduce
Expected behaviour
Pass 'make check' successfully
Actual behaviour
Pass 'make check' unsuccessfully
The text was updated successfully, but these errors were encountered: