[
https://issues.apache.org/jira/browse/HDDS-9524?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Attila Doroszlai updated HDDS-9524:
-----------------------------------
Parent: HDDS-9931
Issue Type: Sub-task (was: Improvement)
> Clean up wait for leader OM in integration tests
> ------------------------------------------------
>
> Key: HDDS-9524
> URL: https://issues.apache.org/jira/browse/HDDS-9524
> Project: Apache Ozone
> Issue Type: Sub-task
> Components: test
> Reporter: Attila Doroszlai
> Assignee: David
> Priority: Major
> Labels: GoodForNewContributors, good-first-issue,
> pull-request-available
> Fix For: 1.5.0
>
>
> {{MiniOzoneHAClusterImpl}}:
> * {{getOMLeader()}} returns current OM leader.
> * {{getOMLeader(boolean)}} optionally waits until OM leader is elected
> There are some tests that implement the same "wait for leader OM" logic using
> the non-waiting {{getOMLeader()}}. Some examples (there may be more!):
> {code}
> hadoop-ozone/integration-test/src/test/java/org/apache/hadoop/ozone/om/TestAddRemoveOzoneManager.java
> 209: GenericTestUtils.waitFor(() -> cluster.getOMLeader() != null, 500,
> 30000);
> 437: GenericTestUtils.waitFor(() -> cluster.getOMLeader() != null, 500,
> 30000);
> hadoop-ozone/integration-test/src/test/java/org/apache/hadoop/ozone/om/TestOzoneManagerHA.java
> 466: getCluster().getOMLeader() != null, 500, timeout);
> hadoop-ozone/integration-test/src/test/java/org/apache/hadoop/ozone/om/TestOzoneManagerHASnapshot.java
> 126: await(120_000, 100, () -> cluster.getOMLeader() != null);
> 253: await(180_000, 100, () -> cluster.getOMLeader() != null);
> {code}
> Goals of this task:
> * rename {{getOMLeader(boolean)}} to {{waitForLeaderOM()}} (the method is
> only called with {{true}})
> * make {{waitForLeaderOM()}} public
> * replace duplicated wait logic in tests with a call to {{waitForLeaderOM()}}
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
