-
Notifications
You must be signed in to change notification settings - Fork 8.8k
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
test for TestServiceInterruptHandling::testRegisterAndRaise #6972
Conversation
💔 -1 overall
This message was automatically generated. |
💔 -1 overall
This message was automatically generated. |
💔 -1 overall
This message was automatically generated. |
💔 -1 overall
This message was automatically generated. |
💔 -1 overall
This message was automatically generated. |
💔 -1 overall
This message was automatically generated. |
💔 -1 overall
This message was automatically generated. |
💔 -1 overall
This message was automatically generated. |
e883400
to
63651c4
Compare
💔 -1 overall
This message was automatically generated. |
…dex order (apache#6813). Contributed by zhengchenyu." This reverts commit e5b76dc.
💔 -1 overall
This message was automatically generated. |
🎊 +1 overall
This message was automatically generated. |
💔 -1 overall
This message was automatically generated. |
💔 -1 overall
This message was automatically generated. |
💔 -1 overall
This message was automatically generated. |
💔 -1 overall
This message was automatically generated. |
💔 -1 overall
This message was automatically generated. |
The reason for the failure of TestServiceInterruptHandling::testRegisterAndRaise has been found. So close this PR. I found SIGINT was ignored before executing TestServiceInterruptHandling::testRegisterAndRaise. In the Java code, for SIGHUP, SIGINT, and SIGTERM, if its SignalHandler have been set to SIG_IGN, any further actions to set SignalHandler will be invalid. For details, see jdk signal In fact, it is difficult for me to reproduce this bug on my current computer. But if you add In order to further prove this problem on docker ci, I submitted commit, then will find the following error like this:
It also proves that SIGINT is ignored. I will fix it in 7008. |
duplicated with #6813, try to test in another server.
For now, this pr only want to fix TestServiceInterruptHandling::testRegisterAndRaise.
I can not reproduce this bug in my environment, even though I have use start-build-env.sh (with centos 7 Dockerfile) to start my test environment. So I have to use ci to test.