You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@geode.apache.org by "Ivan Godwin (Jira)" <ji...@apache.org> on 2020/03/17 17:44:00 UTC

[jira] [Commented] (GEODE-7603) RepeatedRebalanceDUnitTest.testSecondRebalanceIsNotNecessaryWithAddedAndRestartedMembers

    [ https://issues.apache.org/jira/browse/GEODE-7603?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17061078#comment-17061078 ] 

Ivan Godwin commented on GEODE-7603:
------------------------------------

Refactored, but still failing occasionally with following exception:
{code:java}
java.lang.AssertionError: [List check last element] 
Expecting:
 <"0">
not to be equal to:
 <"0">

	at org.apache.geode.management.internal.cli.commands.RepeatedRebalanceDUnitTest.assertPrimariesTransfered(RepeatedRebalanceDUnitTest.java:296)
	at org.apache.geode.management.internal.cli.commands.RepeatedRebalanceDUnitTest.testSecondRebalanceIsNotNecessaryWithAddedAndRestartedMembers(RepeatedRebalanceDUnitTest.java:225)
	at jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
	at jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.lang.reflect.Method.invoke(Method.java:566)
	at org.junit.runners.model.FrameworkMethod$1.runReflectiveCall(FrameworkMethod.java:50)
	at org.junit.internal.runners.model.ReflectiveCallable.run(ReflectiveCallable.java:12)
	at org.junit.runners.model.FrameworkMethod.invokeExplosively(FrameworkMethod.java:47)
	at org.junit.internal.runners.statements.InvokeMethod.evaluate(InvokeMethod.java:17)
	at org.junit.internal.runners.statements.RunBefores.evaluate(RunBefores.java:26)
	at org.junit.internal.runners.statements.RunAfters.evaluate(RunAfters.java:27)
	at org.apache.geode.test.dunit.rules.AbstractDistributedRule$1.evaluate(AbstractDistributedRule.java:59)
	at org.apache.geode.test.junit.rules.DescribedExternalResource$1.evaluate(DescribedExternalResource.java:40)
	at org.apache.geode.test.junit.rules.serializable.SerializableTemporaryFolder$1.evaluate(SerializableTemporaryFolder.java:124)
	at org.junit.rules.RunRules.evaluate(RunRules.java:20)
	at org.junit.runners.ParentRunner.runLeaf(ParentRunner.java:325)
	at org.junit.runners.BlockJUnit4ClassRunner.runChild(BlockJUnit4ClassRunner.java:78)
	at org.junit.runners.BlockJUnit4ClassRunner.runChild(BlockJUnit4ClassRunner.java:57)
	at org.junit.runners.ParentRunner$3.run(ParentRunner.java:290)
	at org.junit.runners.ParentRunner$1.schedule(ParentRunner.java:71)
	at org.junit.runners.ParentRunner.runChildren(ParentRunner.java:288)
	at org.junit.runners.ParentRunner.access$000(ParentRunner.java:58)
	at org.junit.runners.ParentRunner$2.evaluate(ParentRunner.java:268)
	at org.junit.runners.ParentRunner.run(ParentRunner.java:363)
	at org.gradle.api.internal.tasks.testing.junit.JUnitTestClassExecutor.runTestClass(JUnitTestClassExecutor.java:110)
	at org.gradle.api.internal.tasks.testing.junit.JUnitTestClassExecutor.execute(JUnitTestClassExecutor.java:58)
	at org.gradle.api.internal.tasks.testing.junit.JUnitTestClassExecutor.execute(JUnitTestClassExecutor.java:38)
	at org.gradle.api.internal.tasks.testing.junit.AbstractJUnitTestClassProcessor.processTestClass(AbstractJUnitTestClassProcessor.java:62)
	at org.gradle.api.internal.tasks.testing.SuiteTestClassProcessor.processTestClass(SuiteTestClassProcessor.java:51)
	at jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
	at jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.lang.reflect.Method.invoke(Method.java:566)
	at org.gradle.internal.dispatch.ReflectionDispatch.dispatch(ReflectionDispatch.java:35)
	at org.gradle.internal.dispatch.ReflectionDispatch.dispatch(ReflectionDispatch.java:24)
	at org.gradle.internal.dispatch.ContextClassLoaderDispatch.dispatch(ContextClassLoaderDispatch.java:32)
	at org.gradle.internal.dispatch.ProxyDispatchAdapter$DispatchingInvocationHandler.invoke(ProxyDispatchAdapter.java:93)
	at com.sun.proxy.$Proxy2.processTestClass(Unknown Source)
	at org.gradle.api.internal.tasks.testing.worker.TestWorker.processTestClass(TestWorker.java:118)
	at jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
	at jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.lang.reflect.Method.invoke(Method.java:566)
	at org.gradle.internal.dispatch.ReflectionDispatch.dispatch(ReflectionDispatch.java:35)
	at org.gradle.internal.dispatch.ReflectionDispatch.dispatch(ReflectionDispatch.java:24)
	at org.gradle.internal.remote.internal.hub.MessageHubBackedObjectConnection$DispatchWrapper.dispatch(MessageHubBackedObjectConnection.java:175)
	at org.gradle.internal.remote.internal.hub.MessageHubBackedObjectConnection$DispatchWrapper.dispatch(MessageHubBackedObjectConnection.java:157)
	at org.gradle.internal.remote.internal.hub.MessageHub$Handler.run(MessageHub.java:404)
	at org.gradle.internal.concurrent.ExecutorPolicy$CatchAndRecordFailures.onExecute(ExecutorPolicy.java:63)
	at org.gradle.internal.concurrent.ManagedExecutorImpl$1.run(ManagedExecutorImpl.java:46)
	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1128)
	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:628)
	at org.gradle.internal.concurrent.ThreadFactoryImpl$ManagedThreadRunnable.run(ThreadFactoryImpl.java:55)
	at java.lang.Thread.run(Thread.java:834)

{code}
Work has been done on the branch `GEODE-7603-RepeatedRebalanceDUnitTest` of my fork of Geode.
[https://github.com/igodwin/geode/tree/GEODE-7603-RepeatedRebalanceDUnitTest]

I think it would be good to check that members aren't crashing when this failure happens.

> RepeatedRebalanceDUnitTest.testSecondRebalanceIsNotNecessaryWithAddedAndRestartedMembers
> ----------------------------------------------------------------------------------------
>
>                 Key: GEODE-7603
>                 URL: https://issues.apache.org/jira/browse/GEODE-7603
>             Project: Geode
>          Issue Type: Bug
>          Components: tests
>            Reporter: Mark Hanson
>            Assignee: Ivan Godwin
>            Priority: Major
>              Labels: flaky
>
> RepeatedRebalanceDUnitTest.testSecondRebalanceIsNotNecessaryWithAddedAndRestartedMembers failed twice as a part of the mass test run.
> [https://concourse.apachegeode-ci.info/teams/main/pipelines/mhansonp-mhanson-mass-test-ru-main/jobs/DistributedTestOpenJDK8/builds/2160]
>  [https://concourse.apachegeode-ci.info/teams/main/pipelines/mhansonp-mhanson-mass-test-ru-main/jobs/DistributedTestOpenJDK8/builds/2130]
>  
> {noformat}
> 10:56:44org.apache.geode.management.internal.cli.commands.RepeatedRebalanceDUnitTest > testSecondRebalanceIsNotNecessaryWithAddedAndRestartedMembers FAILED
> 10:56:44    java.lang.AssertionError: [List check last element] 
> 10:56:44    Expecting:
> 10:56:44     <"0">
> 10:56:44    not to be equal to:
> 10:56:44     <"0">
> 10:56:44        at org.apache.geode.management.internal.cli.commands.RepeatedRebalanceDUnitTest.assertPrimariesTransfered(RepeatedRebalanceDUnitTest.java:204)
> 10:56:44        at org.apache.geode.management.internal.cli.commands.RepeatedRebalanceDUnitTest.testSecondRebalanceIsNotNecessaryWithAddedAndRestartedMembers(RepeatedRebalanceDUnitTest.java:124)
> 11:06:20
> 11:06:20352 tests completed, 1 failed, 5 skipped
> 11:06:22
> 11:06:22> Task :geode-gfsh:distributedTest FAILED
> 11:54:40
> 11:54:40> Task :geode-core:distributedTest
> 11:54:40{noformat}
>  
> {noformat}
> 22:06:33org.apache.geode.management.internal.cli.commands.RepeatedRebalanceDUnitTest > testSecondRebalanceIsNotNecessaryWithAddedAndRestartedMembers FAILED
> 22:06:33    java.lang.AssertionError: [List check last element] 
> 22:06:33    Expecting:
> 22:06:33     <"0">
> 22:06:33    not to be equal to:
> 22:06:33     <"0">
> 22:06:33        at org.apache.geode.management.internal.cli.commands.RepeatedRebalanceDUnitTest.assertPrimariesTransfered(RepeatedRebalanceDUnitTest.java:204)
> 22:06:33        at org.apache.geode.management.internal.cli.commands.RepeatedRebalanceDUnitTest.testSecondRebalanceIsNotNecessaryWithAddedAndRestartedMembers(RepeatedRebalanceDUnitTest.java:124)
> 22:16:05 {noformat}
>  
>  
>  



--
This message was sent by Atlassian Jira
(v8.3.4#803005)