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

Flaky-test: AdditionalServletsTest.testEmptyStringAsExtractionDirectory #18587

Closed
1 of 2 tasks
codelipenghui opened this issue Nov 24, 2022 · 5 comments · Fixed by #18607
Closed
1 of 2 tasks

Flaky-test: AdditionalServletsTest.testEmptyStringAsExtractionDirectory #18587

codelipenghui opened this issue Nov 24, 2022 · 5 comments · Fixed by #18607

Comments

@codelipenghui
Copy link
Contributor

Search before asking

  • I searched in the issues and found nothing similar.

Example failure

https://github.com/apache/pulsar/actions/runs/3530823792/jobs/5923383709

Exception stacktrace

2022-11-23T09:52:32.2426617Z [ERROR] Tests run: 2, Failures: 1, Errors: 0, Skipped: 1, Time elapsed: 0.418 s <<< FAILURE! - in org.apache.pulsar.broker.web.plugin.servlet.AdditionalServletsTest
2022-11-23T09:52:32.2427429Z [ERROR] testEmptyStringAsExtractionDirectory(org.apache.pulsar.broker.web.plugin.servlet.AdditionalServletsTest)  Time elapsed: 0.018 s  <<< FAILURE!
2022-11-23T09:52:32.2428297Z java.lang.NullPointerException: Cannot invoke "org.apache.pulsar.broker.web.plugin.servlet.AdditionalServletDefinitions.servlets()" because "definitions" is null
2022-11-23T09:52:32.2429066Z 	at org.apache.pulsar.broker.web.plugin.servlet.AdditionalServlets.load(AdditionalServlets.java:89)
2022-11-23T09:52:32.2429916Z 	at org.apache.pulsar.broker.web.plugin.servlet.AdditionalServletsTest.testEmptyStringAsExtractionDirectory(AdditionalServletsTest.java:64)
2022-11-23T09:52:32.2430669Z 	at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
2022-11-23T09:52:32.2431238Z 	at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:77)
2022-11-23T09:52:32.2431862Z 	at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
2022-11-23T09:52:32.2432372Z 	at java.base/java.lang.reflect.Method.invoke(Method.java:568)
2022-11-23T09:52:32.2432886Z 	at org.testng.internal.invokers.MethodInvocationHelper.invokeMethod(MethodInvocationHelper.java:139)
2022-11-23T09:52:32.2433496Z 	at org.testng.internal.invokers.InvokeMethodRunnable.runOne(InvokeMethodRunnable.java:47)
2022-11-23T09:52:32.2434054Z 	at org.testng.internal.invokers.InvokeMethodRunnable.call(InvokeMethodRunnable.java:76)
2022-11-23T09:52:32.2434590Z 	at org.testng.internal.invokers.InvokeMethodRunnable.call(InvokeMethodRunnable.java:11)
2022-11-23T09:52:32.2435071Z 	at java.base/java.util.concurrent.FutureTask.run(FutureTask.java:264)
2022-11-23T09:52:32.2435553Z 	at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1136)
2022-11-23T09:52:32.2436078Z 	at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:635)
2022-11-23T09:52:32.2436483Z 	at java.base/java.lang.Thread.run(Thread.java:833)

Are you willing to submit a PR?

  • I'm willing to submit a PR!
@poorbarcode
Copy link
Contributor

@lhotari
Copy link
Member

lhotari commented Nov 24, 2022

@lhotari
Copy link
Member

lhotari commented Nov 24, 2022

@andrasbeni Is this related to #17477 ?

@andrasbeni
Copy link
Contributor

@lhotari, yes, it certainly is. See above my attempt at fixing it.

@lhotari
Copy link
Member

lhotari commented Nov 24, 2022

thank you @andrasbeni for addressing this so quickly!

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