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

[ FAIL ] kj/async-io-test.c++:307: legacy test: AsyncIo/AncillaryMessageHandler (854 μs) #1349

Closed
vt-alt opened this issue Oct 1, 2021 · 6 comments

Comments

@vt-alt
Copy link

vt-alt commented Oct 1, 2021

When building capnproto v0.9.1 for the package for ALT Linux we got such make check error:

[ TEST ] kj/async-io-test.c++:307: legacy test: AsyncIo/AncillaryMessageHandler
kj/async-io-test.c++:364: failed: expected serverHandlerCalled
stack: 55b5b64c4bfd 55b5b627b2f9 7fcd39c0950d 7fcd39bbc4d9 7fcd39c09a7c 7fcd39c0a2b1 7fcd39bdea4d 7fcd39bbc4d
9 7fcd39bd7f91 7fcd39c087b0 7fcd39708e2c 55b5b60ba0c9
[ FAIL ] kj/async-io-test.c++:307: legacy test: AsyncIo/AncillaryMessageHandler (854 μs)
...
907 test(s) passed 
1 test(s) failed 
FAIL: capnp-test
Randomly testing backwards-compatibility scenarios with seed: 1633094793
PASS: capnp-evolution-test
PASS: src/capnp/compiler/capnp-test.sh
===========================================
1 of 3 tests failed

Please ask if you need any additional info.

@kentonv
Copy link
Member

kentonv commented Oct 4, 2021

That's odd. What kernel version?

@vt-alt
Copy link
Author

vt-alt commented Oct 4, 2021

Tested on 4.9.56.

@kentonv
Copy link
Member

kentonv commented Oct 4, 2021

I'm not sure what to say here. Seems like your kernel is behaving unusually. You can safely ignore the test failure as this particular functionality isn't really used by Cap'n Proto itself.

@vt-alt
Copy link
Author

vt-alt commented Oct 7, 2021

Thanks. Maybe we could reproduce the test with this line resolved

stack: 55b5b64c4bfd 55b5b627b2f9 7fcd39c0950d 7fcd39bbc4d9 7fcd39c09a7c 7fcd39c0a2b1 7fcd39bdea4d 7fcd39bbc4d
9 7fcd39bd7f91 7fcd39c087b0 7fcd39708e2c 55b5b60ba0c9

if you suggest how?

But, if this really does not matter, could you suggest the way to skip this test? I am not familiar with your test harness, but would like to leave make check run on package build.

@kentonv
Copy link
Member

kentonv commented Oct 8, 2021

Sorry, the stack trace probably doesn't help me (even if I could decode it). This probably needs to be debugged directly on a system exhibiting the problem.

Do you have the ability to float patches on your packages? If so I'd suggest adding #if 0 around this test case. Otherwise, I'd accept a PR which adds something like #if KJ_SKIP_CMSG_HANDLER_TEST around that test, and then you can add that to your cflags when you build...

@vt-alt
Copy link
Author

vt-alt commented Oct 9, 2021

Thanks for explanations! I did sed -i '/TEST(AsyncIo, AncillaryMessageHandler)/,/^}/s/^/\/\//' c++/src/kj/async-io-test.c++ .

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

No branches or pull requests

2 participants