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

Viraj Jasani commented on HDFS-16336:
-------------------------------------

We have two options for 3.3 and 3.2:
 # We can backport GenericTestUtils new method from HDFS-16171
 # For 3.3 and 3.2 backport patch of this main PR, we can remove last argument 
"RollingUpgradeStatus is already set" from waitForNullMxBean()'s call to 
GenericTestUtils.waitFor()

While 1) might not be direct backport and might require PR, 2) should be quite 
simplified.

[~ste...@apache.org] [~aajisaka] to unblock 3.3 and 3.2 builds quickly, could 
you please commit option 2) from the above? Here is the patch:
{code:java}
--- 
a/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/TestRollingUpgrade.java
+++ 
b/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/TestRollingUpgrade.java
@@ -366,7 +366,7 @@ private void waitForNullMxBean() throws TimeoutException, 
InterruptedException {
         LOG.error("Something went wrong.", t);
         return false;
       }
-    }, 100, 8000, "RollingUpgradeStatus is already set");
+    }, 100, 8000);
   }
 
   private static void startRollingUpgrade(Path foo, Path bar, {code}
If you also prefer option 1), I can create backport PRs for HDFS-16171 but in 
the meantime, above patch will unblock builds for both 3.3 and 3.2.

Thanks

> De-flake TestRollingUpgrade#testRollback
> ----------------------------------------
>
>                 Key: HDFS-16336
>                 URL: https://issues.apache.org/jira/browse/HDFS-16336
>             Project: Hadoop HDFS
>          Issue Type: Sub-task
>          Components: hdfs, test
>    Affects Versions: 3.4.0
>            Reporter: Kevin Wikant
>            Assignee: Viraj Jasani
>            Priority: Minor
>              Labels: pull-request-available
>             Fix For: 3.4.0, 3.2.4, 3.3.3
>
>          Time Spent: 2h
>  Remaining Estimate: 0h
>
> This pull request: [https://github.com/apache/hadoop/pull/3675]
> Failed Jenkins pre-commit job due to an unrelated unit test failure: 
> [https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3675/1/artifact/out/patch-unit-hadoop-hdfs-project_hadoop-hdfs.txt]
> {code:java}
> [ERROR] Failures: 
> [ERROR] 
> org.apache.hadoop.hdfs.TestRollingUpgrade.testRollback(org.apache.hadoop.hdfs.TestRollingUpgrade)
> [ERROR]   Run 1: TestRollingUpgrade.testRollback:328->checkMxBeanIsNull:299 
> expected null, but 
> was:<javax.management.openmbean.CompositeDataSupport(compositeType=javax.management.openmbean.CompositeType(name=org.apache.hadoop.hdfs.protocol.RollingUpgradeInfo$Bean,items=((itemName=blockPoolId,itemType=javax.management.openmbean.SimpleType(name=java.lang.String)),(itemName=createdRollbackImages,itemType=javax.management.openmbean.SimpleType(name=java.lang.Boolean)),(itemName=finalizeTime,itemType=javax.management.openmbean.SimpleType(name=java.lang.Long)),(itemName=startTime,itemType=javax.management.openmbean.SimpleType(name=java.lang.Long)))),contents={blockPoolId=BP-1039609189-172.17.0.2-1637204447583,
>  createdRollbackImages=true, finalizeTime=0, startTime=1637204448659})>
> [ERROR]   Run 2: TestRollingUpgrade.testRollback:328->checkMxBeanIsNull:299 
> expected null, but 
> was:<javax.management.openmbean.CompositeDataSupport(compositeType=javax.management.openmbean.CompositeType(name=org.apache.hadoop.hdfs.protocol.RollingUpgradeInfo$Bean,items=((itemName=blockPoolId,itemType=javax.management.openmbean.SimpleType(name=java.lang.String)),(itemName=createdRollbackImages,itemType=javax.management.openmbean.SimpleType(name=java.lang.Boolean)),(itemName=finalizeTime,itemType=javax.management.openmbean.SimpleType(name=java.lang.Long)),(itemName=startTime,itemType=javax.management.openmbean.SimpleType(name=java.lang.Long)))),contents={blockPoolId=BP-1039609189-172.17.0.2-1637204447583,
>  createdRollbackImages=true, finalizeTime=0, startTime=1637204448659})>
> [ERROR]   Run 3: TestRollingUpgrade.testRollback:328->checkMxBeanIsNull:299 
> expected null, but 
> was:<javax.management.openmbean.CompositeDataSupport(compositeType=javax.management.openmbean.CompositeType(name=org.apache.hadoop.hdfs.protocol.RollingUpgradeInfo$Bean,items=((itemName=blockPoolId,itemType=javax.management.openmbean.SimpleType(name=java.lang.String)),(itemName=createdRollbackImages,itemType=javax.management.openmbean.SimpleType(name=java.lang.Boolean)),(itemName=finalizeTime,itemType=javax.management.openmbean.SimpleType(name=java.lang.Long)),(itemName=startTime,itemType=javax.management.openmbean.SimpleType(name=java.lang.Long)))),contents={blockPoolId=BP-1039609189-172.17.0.2-1637204447583,
>  createdRollbackImages=true, finalizeTime=0, startTime=1637204448659})> {code}
> Seems that perhaps "TestRollingUpgrade.testRollback" is a flaky unit test



--
This message was sent by Atlassian Jira
(v8.20.1#820001)

---------------------------------------------------------------------
To unsubscribe, e-mail: hdfs-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: hdfs-issues-h...@hadoop.apache.org

Reply via email to