Fixed: https://github.com/apache/incubator-brooklyn/pull/1058
Verified by running the test in question hundreds of times in a loop -- before the fix it would always end up failing out of the loop, but now it always makes it through alive and well. On Fri, Nov 20, 2015 at 12:09 PM Mike Zaccardo < [email protected]> wrote: > I added cluster member IDs and authored this test. I will look into it. > > -Mike > > On Fri, Nov 20, 2015, 06:50 David Lloyd <[email protected]> > wrote: > >> I saw this last Friday (the 13th!) as well. Happened once when using >> maven >> on the command line; then tried running maven from IDEA and the problem >> didn't occur; finally tried the command line directly again and still it >> didn't appear. >> >> Agree that this is probably a race condition problem. >> >> /Dave. >> >> On 20 November 2015 at 14:37, Svetoslav Neykov < >> [email protected]> wrote: >> >> > Haven't looked in the test, but the default logic is to remove the >> newest >> > members. Not clear about the addition order. >> > >> > Svet. >> > >> > >> > > On 20.11.2015 г., at 16:36, Martin Harris < >> > [email protected]> wrote: >> > > >> > > Hi Folks, >> > > >> > > I'm getting intermittent failure of the >> > > >> > >> `DynamicClusterTest.testAllClusterMemberIdsAddedInOrderOnNegativeThenPositiveResize`. >> > > It seems to be a race condition related to how the entities are >> numbered >> > > >> > > Does anyone know if we can actually guarantee the numbers should be >> > > allocated in the way that the test assumes, or is the test in error? >> > > >> > > Cheers >> > > >> > > >> > > [1]: >> > > >> > >> https://github.com/apache/incubator-brooklyn/commit/cea346284a9c0d2556f2ffd7ee2fd033c4e4adbd >> > > >> > > -- >> > > Martin Harris >> > > Lead Software Engineer >> > > Cloudsoft Corporation Ltd >> > > www.cloudsoftcorp.com >> > > Mobile: +44 (0)7989 047-855 >> > >> > >> >
