You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pulsar.apache.org by "lhotari (via GitHub)" <gi...@apache.org> on 2023/02/02 12:40:00 UTC
[GitHub] [pulsar] lhotari opened a new issue, #19401: Flaky-test: BrokerRegistryTest.testRegisterAndLookup
lhotari opened a new issue, #19401:
URL: https://github.com/apache/pulsar/issues/19401
### Search before asking
- [X] I searched in the [issues](https://github.com/apache/pulsar/issues) and found nothing similar.
### Example failure
https://github.com/apache/pulsar/actions/runs/4073474341/jobs/7019289431#step:11:1397
### Exception stacktrace
```
Error: Tests run: 12, Failures: 1, Errors: 0, Skipped: 8, Time elapsed: 18.168 s <<< FAILURE! - in org.apache.pulsar.broker.loadbalance.extensions.BrokerRegistryTest
Error: testRegisterAndLookup(org.apache.pulsar.broker.loadbalance.extensions.BrokerRegistryTest) Time elapsed: 5.181 s <<< FAILURE!
java.lang.AssertionError: expected [2] but found [3]
at org.testng.Assert.fail(Assert.java:110)
at org.testng.Assert.failNotEquals(Assert.java:1413)
at org.testng.Assert.assertEqualsImpl(Assert.java:149)
at org.testng.Assert.assertEquals(Assert.java:131)
at org.testng.Assert.assertEquals(Assert.java:1240)
at org.testng.Assert.assertEquals(Assert.java:1274)
at org.apache.pulsar.broker.loadbalance.extensions.BrokerRegistryTest.testRegisterAndLookup(BrokerRegistryTest.java:277)
at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:77)
at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
at java.base/java.lang.reflect.Method.invoke(Method.java:568)
at org.testng.internal.invokers.MethodInvocationHelper.invokeMethod(MethodInvocationHelper.java:139)
at org.testng.internal.invokers.InvokeMethodRunnable.runOne(InvokeMethodRunnable.java:47)
at org.testng.internal.invokers.InvokeMethodRunnable.call(InvokeMethodRunnable.java:76)
at org.testng.internal.invokers.InvokeMethodRunnable.call(InvokeMethodRunnable.java:11)
at java.base/java.util.concurrent.FutureTask.run(FutureTask.java:264)
at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1136)
at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:635)
at java.base/java.lang.Thread.run(Thread.java:833)
```
### Are you willing to submit a PR?
- [ ] I'm willing to submit a PR!
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [pulsar] lhotari commented on issue #19401: Flaky-test: BrokerRegistryTest.testRegisterAndLookup
Posted by "lhotari (via GitHub)" <gi...@apache.org>.
lhotari commented on issue #19401:
URL: https://github.com/apache/pulsar/issues/19401#issuecomment-1413684486
@Demogorgon314 @heesung-sn Would you be able to fix this flaky test? It looks like it was introduced in #18810
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [pulsar] lhotari closed issue #19401: Flaky-test: BrokerRegistryTest.testRegisterAndLookup
Posted by "lhotari (via GitHub)" <gi...@apache.org>.
lhotari closed issue #19401: Flaky-test: BrokerRegistryTest.testRegisterAndLookup
URL: https://github.com/apache/pulsar/issues/19401
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [pulsar] lhotari commented on issue #19401: Flaky-test: BrokerRegistryTest.testRegisterAndLookup
Posted by "lhotari (via GitHub)" <gi...@apache.org>.
lhotari commented on issue #19401:
URL: https://github.com/apache/pulsar/issues/19401#issuecomment-1413693917
@Demogorgon314 Another problem is that the test leaves the test Pulsar brokers running.
This code is not valid for tests:
https://github.com/apache/pulsar/blob/8a4a8a63c2fbb18001594bd9ce91142b1cd44219/pulsar-broker/src/test/java/org/apache/pulsar/broker/loadbalance/extensions/BrokerRegistryTest.java#L214-L216
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [pulsar] lhotari commented on issue #19401: Flaky-test: BrokerRegistryTest.testRegisterAndLookup
Posted by "lhotari (via GitHub)" <gi...@apache.org>.
lhotari commented on issue #19401:
URL: https://github.com/apache/pulsar/issues/19401#issuecomment-1413695877
@Demogorgon314 I added the full logs for the failed test run to the description of the issue.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [pulsar] lhotari commented on issue #19401: Flaky-test: BrokerRegistryTest.testRegisterAndLookup
Posted by "lhotari (via GitHub)" <gi...@apache.org>.
lhotari commented on issue #19401:
URL: https://github.com/apache/pulsar/issues/19401#issuecomment-1413710282
@Demogorgon314 Yet another failure: https://github.com/apache/pulsar/actions/runs/4073367805/jobs/7019320477#step:11:1395 . It seems that this is very flaky. Logs: https://gist.github.com/lhotari/cb90c0eeb0320920be512c03f0258434
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [pulsar] Demogorgon314 commented on issue #19401: Flaky-test: BrokerRegistryTest.testRegisterAndLookup
Posted by "Demogorgon314 (via GitHub)" <gi...@apache.org>.
Demogorgon314 commented on issue #19401:
URL: https://github.com/apache/pulsar/issues/19401#issuecomment-1413713004
Thanks! I have found the root cause, and I'll push a PR to fix it soon.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org