You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@geode.apache.org by "Bill Burcham (Jira)" <ji...@apache.org> on 2020/10/12 17:59:00 UTC
[jira] [Assigned] (GEODE-8602)
locatorsStopWaitingForLocatorWaitTimeIfAllLocatorsContacted fails because
test doesn't wait for view to have all members
[ https://issues.apache.org/jira/browse/GEODE-8602?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Bill Burcham reassigned GEODE-8602:
-----------------------------------
Assignee: Dan Smith
> locatorsStopWaitingForLocatorWaitTimeIfAllLocatorsContacted fails because test doesn't wait for view to have all members
> ------------------------------------------------------------------------------------------------------------------------
>
> Key: GEODE-8602
> URL: https://issues.apache.org/jira/browse/GEODE-8602
> Project: Geode
> Issue Type: Bug
> Components: membership, tests
> Affects Versions: 1.14.0
> Reporter: Bill Burcham
> Assignee: Dan Smith
> Priority: Major
>
> {{MembershipIntegrationTest.locatorsStopWaitingForLocatorWaitTimeIfAllLocatorsContacted()}} correctly waits for locators to start but then fails to wait for locators' views to be updated.
> This code:
> {code:java}
> assertThat(membership0.getView().getMembers()).hasSize(2);
> {code}
> …doesn't wait. This results in an exception sometimes:
> {code:java}
> org.apache.geode.distributed.internal.membership.gms.MembershipIntegrationTest > locatorsStopWaitingForLocatorWaitTimeIfAllLocatorsContacted FAILED
> java.lang.AssertionError:
> Expected size:<2> but was:<1> in:
> <[172.17.0.30(1:locator)<ec><v0>:41000]>
> at org.apache.geode.distributed.internal.membership.gms.MembershipIntegrationTest.locatorsStopWaitingForLocatorWaitTimeIfAllLocatorsContacted(MembershipIntegrationTest.java:308)
> {code}
> A Band-Aid would be to put an awaitility call around the two such hasSize() calls. A better fix would be to make the test deterministic per GEODE-8589.
--
This message was sent by Atlassian Jira
(v8.3.4#803005)