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

ASF GitHub Bot commented on GEODE-8609:
---------------------------------------

jdeppe-pivotal commented on a change in pull request #5625:
URL: https://github.com/apache/geode/pull/5625#discussion_r505719069



##########
File path: 
geode-dunit/src/main/java/org/apache/geode/test/dunit/internal/DUnitLauncher.java
##########
@@ -395,12 +395,18 @@ public static void closeAndCheckForSuspects() {
           + "' - perhaps a rule that is cleaning up before suspect processing 
has already run.");
     }
 
+    boolean ok = true;

Review comment:
       Good idea - I'll apply that shortly.




----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


> Create a dunit suspect file per VM
> ----------------------------------
>
>                 Key: GEODE-8609
>                 URL: https://issues.apache.org/jira/browse/GEODE-8609
>             Project: Geode
>          Issue Type: Test
>          Components: tests
>            Reporter: Jens Deppe
>            Assignee: Jens Deppe
>            Priority: Major
>              Labels: pull-request-available
>
> In some situations, especially when restarting Dunit VMs, it appears that the 
> suspect log file can be corrupted since suspect processing reports errors 
> such as:
> {noformat}
> org.apache.geode.redis.internal.executor.CrashAndNoRepeatDUnitTest > 
> classMethod FAILED
>     java.lang.AssertionError: Suspicious strings were written to the log 
> during this run.
>     Fix the strings or use IgnoredException.addIgnoredException to ignore.
>     -----------------------------------------------------------------------
>     Found suspect string in log4j at line 19594      
> /home/geode/.gradle/caches/modules-2/files-2.1/antlr/antlr/2.7.7/83cd2cd674a21[warn
>  2020/10/11 15:23:24.988 GMT <GeodeRedisServer-Command-2> tid=105] Execution 
> of Redis command APPEND append-key-1 -append-key-1-17484-  failed: 
> org.apache.geode.cache.execute.FunctionException: 
> org.apache.geode.cache.execute.FunctionInvocationTargetException: 
> memberDeparted event for < 172.17.0.20(server-3:2004)<v15>:41003 > crashed, 
> true{noformat}
> Where the suspected string is corrupted.
> The proposed fix will create a new dunit_suspect log for each VM. The logs 
> will now be named {{dunit_suspect-vm<VMID>.log}}. The {{locator}} vm and the 
> test runner vm will have logs named {{dunit_suspect-locator.log}} and 
> {{dunit_suspect-local.log}} respectively.



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

Reply via email to