I tried to revert just a few of the commits. That didn't work so now I'm testing a branch where I've reverted all of the commits I pulled in yesterday. Hope to have that ready later this morning.
On Fri, Aug 11, 2017 at 8:13 PM, Kirk Lund <kl...@apache.org> wrote: > I merged in 16 PRs today and precheckin was clean for each individually. I > launched a precheckin after all were merged in and that seems to have tons > of Management test failures. So on Monday, I'll try to identify which PRs > aren't playing nicely with each other and start backing them out. Sorry the > tests are going to be so messy for the weekend. > > I think this is one of the times where trying to separate a huge > refactoring out into many tickets just didn't work. Sorry code reviewers, > but some of these refactorings are going to have to be HUGE change-sets. > > :geode-core:integrationTest > > org.apache.geode.management.internal.cli.help.HelperIntegrationTest > > testHelpWithNoInput FAILED > java.lang.AssertionError: > Expected size:<2> but was:<4> in: > <["help (Available)", > "Display syntax and usage information for all commands or list all > available commands if <command> isn't specified.", > "hint (Available)", > "Provide hints for a topic or list all available topics if "topic" > isn't specified."]> > at org.apache.geode.management.internal.cli.help. > HelperIntegrationTest.testHelpWithNoInput(HelperIntegrationTest.java:69) > > org.apache.geode.management.internal.cli.help.HelperIntegrationTest > > testHintWithNoInput FAILED > java.lang.AssertionError: > Expected size:<21> but was:<22> in: > <["Hints are available for the following topics. Use "hint > <topic-name>" for a specific hint.", > "", > "Client", > "Cluster Configuration", > "Configuration", > "Data", > "Debug-Utility", > "Disk Store", > "Function Execution", > "GFSH", > "Geode", > "Help", > "JMX", > "Lifecycle", > "Locator", > "Logs", > "Management-Monitoring", > "Manager", > "Region", > "Server", > "Statistics", > "WAN"]> > at org.apache.geode.management.internal.cli.help. > HelperIntegrationTest.testHintWithNoInput(HelperIntegrationTest.java:96) > > org.apache.geode.management.internal.cli.GfshParserConverterTest > > testHintConverter FAILED > java.lang.StringIndexOutOfBoundsException: String index out of range: > -1 > at java.lang.String.substring(String.java:1931) > at org.apache.geode.management.internal.cli.GfshParser. > lambda$completeAdvanced$0(GfshParser.java:265) > at java.util.ArrayList.replaceAll(ArrayList.java:1442) > at org.apache.geode.management.internal.cli.GfshParser. > completeAdvanced(GfshParser.java:264) > at org.apache.geode.test.dunit.rules.GfshParserRule.complete( > GfshParserRule.java:58) > at org.apache.geode.management.internal.cli. > GfshParserConverterTest.testHintConverter(GfshParserConverterTest.java: > 126) > > org.apache.geode.management.internal.security.CliCommandsSecurityTest > > testNoAccess FAILED > org.assertj.core.api.SoftAssertionError: > The following assertion failed: > 1) [destroy function --id=InterestCalculations] > Expecting: > <"stranger not authorized for CLUSTER:MANAGE:JAR"> > to contain: > <"DATA:MANAGE"> > at CliCommandsSecurityTest.testNoAccess(CliCommandsSecurityTest.java: > 74) > at org.assertj.core.api.SoftAssertions.assertAll( > SoftAssertions.java:134) > at org.apache.geode.management.internal.security. > CliCommandsSecurityTest.testNoAccess(CliCommandsSecurityTest.java:78) > > org.apache.geode.management.internal.security.GfshCommandsSecurityTest > > testRegionAReader FAILED > org.junit.ComparisonFailure: [destroy function > --id=InterestCalculations] expected:<[110]> but was:<[415]> > at sun.reflect.NativeConstructorAccessorImpl.newInstance0(Native > Method) > at sun.reflect.NativeConstructorAccessorImpl.newInstance( > NativeConstructorAccessorImpl.java:62) > at sun.reflect.DelegatingConstructorAccessorImpl.newInstance( > DelegatingConstructorAccessorImpl.java:45) > at org.apache.geode.management.internal.security. > GfshCommandsSecurityTest.runCommandsPermittedAndForbidd > enBy(GfshCommandsSecurityTest.java:164) > at org.apache.geode.management.internal.security. > GfshCommandsSecurityTest.testRegionAReader(GfshCommandsSecurityTest.java: > 108) > > org.apache.geode.management.internal.security.GfshCommandsSecurityTest > > testRegionAWriter FAILED > org.junit.ComparisonFailure: [destroy function > --id=InterestCalculations] expected:<[110]> but was:<[415]> > at sun.reflect.NativeConstructorAccessorImpl.newInstance0(Native > Method) > at sun.reflect.NativeConstructorAccessorImpl.newInstance( > NativeConstructorAccessorImpl.java:62) > at sun.reflect.DelegatingConstructorAccessorImpl.newInstance( > DelegatingConstructorAccessorImpl.java:45) > at org.apache.geode.management.internal.security. > GfshCommandsSecurityTest.runCommandsPermittedAndForbidd > enBy(GfshCommandsSecurityTest.java:164) > at org.apache.geode.management.internal.security. > GfshCommandsSecurityTest.testRegionAWriter(GfshCommandsSecurityTest.java: > 114) > > org.apache.geode.management.internal.security.GfshCommandsSecurityTest > > testClusterReader FAILED > org.junit.ComparisonFailure: [destroy function > --id=InterestCalculations] expected:<[110]> but was:<[415]> > at sun.reflect.NativeConstructorAccessorImpl.newInstance0(Native > Method) > at sun.reflect.NativeConstructorAccessorImpl.newInstance( > NativeConstructorAccessorImpl.java:62) > at sun.reflect.DelegatingConstructorAccessorImpl.newInstance( > DelegatingConstructorAccessorImpl.java:45) > at org.apache.geode.management.internal.security. > GfshCommandsSecurityTest.runCommandsPermittedAndForbidd > enBy(GfshCommandsSecurityTest.java:164) > at org.apache.geode.management.internal.security. > GfshCommandsSecurityTest.testClusterReader(GfshCommandsSecurityTest.java: > 72) > > org.apache.geode.management.internal.security.GfshCommandsSecurityTest > > testClusterWriter FAILED > org.junit.ComparisonFailure: [destroy function > --id=InterestCalculations] expected:<[110]> but was:<[415]> > at sun.reflect.NativeConstructorAccessorImpl.newInstance0(Native > Method) > at sun.reflect.NativeConstructorAccessorImpl.newInstance( > NativeConstructorAccessorImpl.java:62) > at sun.reflect.DelegatingConstructorAccessorImpl.newInstance( > DelegatingConstructorAccessorImpl.java:45) > at org.apache.geode.management.internal.security. > GfshCommandsSecurityTest.runCommandsPermittedAndForbidd > enBy(GfshCommandsSecurityTest.java:164) > at org.apache.geode.management.internal.security. > GfshCommandsSecurityTest.testClusterWriter(GfshCommandsSecurityTest.java: > 78) > > org.apache.geode.management.internal.security.GfshCommandsSecurityTest > > testClusterManager FAILED > org.junit.ComparisonFailure: [destroy function > --id=InterestCalculations] expected:<[110]> but was:<[415]> > at sun.reflect.NativeConstructorAccessorImpl.newInstance0(Native > Method) > at sun.reflect.NativeConstructorAccessorImpl.newInstance( > NativeConstructorAccessorImpl.java:62) > at sun.reflect.DelegatingConstructorAccessorImpl.newInstance( > DelegatingConstructorAccessorImpl.java:45) > at org.apache.geode.management.internal.security. > GfshCommandsSecurityTest.runCommandsPermittedAndForbidd > enBy(GfshCommandsSecurityTest.java:164) > at org.apache.geode.management.internal.security. > GfshCommandsSecurityTest.testClusterManager(GfshCommandsSecurityTest.java: > 84) > > org.apache.geode.management.internal.security.GfshCommandsSecurityTest > > testDataReader FAILED > org.junit.ComparisonFailure: [destroy function > --id=InterestCalculations] expected:<[110]> but was:<[415]> > at sun.reflect.NativeConstructorAccessorImpl.newInstance0(Native > Method) > at sun.reflect.NativeConstructorAccessorImpl.newInstance( > NativeConstructorAccessorImpl.java:62) > at sun.reflect.DelegatingConstructorAccessorImpl.newInstance( > DelegatingConstructorAccessorImpl.java:45) > at org.apache.geode.management.internal.security. > GfshCommandsSecurityTest.runCommandsPermittedAndForbidd > enBy(GfshCommandsSecurityTest.java:164) > at org.apache.geode.management.internal.security. > GfshCommandsSecurityTest.testDataReader(GfshCommandsSecurityTest.java:90) > > org.apache.geode.management.internal.security.GfshCommandsSecurityTest > > testRegionAManager FAILED > org.junit.ComparisonFailure: [destroy function > --id=InterestCalculations] expected:<[110]> but was:<[415]> > at sun.reflect.NativeConstructorAccessorImpl.newInstance0(Native > Method) > at sun.reflect.NativeConstructorAccessorImpl.newInstance( > NativeConstructorAccessorImpl.java:62) > at sun.reflect.DelegatingConstructorAccessorImpl.newInstance( > DelegatingConstructorAccessorImpl.java:45) > at org.apache.geode.management.internal.security. > GfshCommandsSecurityTest.runCommandsPermittedAndForbidd > enBy(GfshCommandsSecurityTest.java:164) > at org.apache.geode.management.internal.security. > GfshCommandsSecurityTest.testRegionAManager(GfshCommandsSecurityTest.java: > 120) > > org.apache.geode.management.internal.security.GfshCommandsSecurityTest > > testDataWriter FAILED > org.junit.ComparisonFailure: [destroy function > --id=InterestCalculations] expected:<[110]> but was:<[415]> > at sun.reflect.NativeConstructorAccessorImpl.newInstance0(Native > Method) > at sun.reflect.NativeConstructorAccessorImpl.newInstance( > NativeConstructorAccessorImpl.java:62) > at sun.reflect.DelegatingConstructorAccessorImpl.newInstance( > DelegatingConstructorAccessorImpl.java:45) > at org.apache.geode.management.internal.security. > GfshCommandsSecurityTest.runCommandsPermittedAndForbidd > enBy(GfshCommandsSecurityTest.java:164) > at org.apache.geode.management.internal.security. > GfshCommandsSecurityTest.testDataWriter(GfshCommandsSecurityTest.java:96) > > org.apache.geode.codeAnalysis.AnalyzeSerializablesJUnitTest > > testSerializables FAILED > java.lang.AssertionError: New or moved classes----------------------- > ----------------- > org/apache/geode/internal/cache/snapshot/ParallelSnapshotFileMapper, > true,1 > > > If the class is not persisted or sent over the wire add it to the file > /tmp/build/ae3c03f4/gemfire/open/geode-core/src/test/ > resources/org/apache/geode/codeAnalysis/excludedClasses.txt > Otherwise if this doesn't break backward compatibility, copy the file > > /tmp/build/ae3c03f4/geode/geode-core/build/integrationTest/actualSerializables.dat > to > /tmp/build/ae3c03f4/gemfire/open/geode-core/src/test/ > resources/org/apache/geode/codeAnalysis/sanctionedSerializables.txt. > at org.apache.geode.codeAnalysis.AnalyzeSerializablesJUnitTest. > testSerializables(AnalyzeSerializablesJUnitTest.java:150) > > 3715 tests completed, 13 failed, 140 skipped > :geode-core:integrationTest FAILED > > :geode-web:integrationTest > org.apache.geode.management.internal.security.GfshCommandsOverHttpSecurityTest > > testRegionAReader FAILED > org.junit.ComparisonFailure: [destroy function --id=InterestCalculations] > expected:<[110]> but was:<[415]> > org.apache.geode.management.internal.security.GfshCommandsOverHttpSecurityTest > > testRegionAWriter FAILED > org.junit.ComparisonFailure: [destroy function --id=InterestCalculations] > expected:<[110]> but was:<[415]> > org.apache.geode.management.internal.security.GfshCommandsOverHttpSecurityTest > > testClusterReader FAILED > org.junit.ComparisonFailure: [destroy function --id=InterestCalculations] > expected:<[110]> but was:<[415]> > org.apache.geode.management.internal.security.GfshCommandsOverHttpSecurityTest > > testClusterWriter FAILED > org.junit.ComparisonFailure: [destroy function --id=InterestCalculations] > expected:<[110]> but was:<[415]> > org.apache.geode.management.internal.security.GfshCommandsOverHttpSecurityTest > > testClusterManager FAILED > org.junit.ComparisonFailure: [destroy function --id=InterestCalculations] > expected:<[110]> but was:<[415]> > org.apache.geode.management.internal.security.GfshCommandsOverHttpSecurityTest > > testDataReader FAILED > org.junit.ComparisonFailure: [destroy function --id=InterestCalculations] > expected:<[110]> but was:<[415]> > org.apache.geode.management.internal.security.GfshCommandsOverHttpSecurityTest > > testRegionAManager FAILED > org.junit.ComparisonFailure: [destroy function --id=InterestCalculations] > expected:<[110]> but was:<[415]> > org.apache.geode.management.internal.security.GfshCommandsOverHttpSecurityTest > > testDataWriter FAILED > org.junit.ComparisonFailure: [destroy function --id=InterestCalculations] > expected:<[110]> but was:<[415]> > 30 tests completed, 8 failed > :geode-web:integrationTest FAILED > >