[GitHub] [helix] xyuanlu edited a comment on issue #1336: Flaky test testUpdateConfigFields

[GitBox <gi...@apache.org>]

xyuanlu edited a comment on issue #1336:
URL: https://github.com/apache/helix/issues/1336#issuecomment-733908737


   The test is constantly failing on master now. The direct reason is the same, there is an empty `INSTANCE_CAPACITY_KEYS=[],` list so we get a IndexOutOfBoundsException error. 
   
   My theory is that, this empty list could be added by TestInstancesAccessor.testValidateWeightForAllInstances() or TestPerInstanceAccessor.testValidateWeightForInstance() since these two tests and TestClusterAccessor.testUpdateConfigFields() uses the same cluster "TestCluster_0". 
   
   In the test log where testUpdateConfigFields is failing, the two tests that added this empty list always happen before this testUpdateConfigFields.  
   
   My solution is simply changing the cluster for testUpdateConfigFields to another test cluster and use "TestCluster_0" only for WAGED related tests. 


----------------------------------------------------------------
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:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@helix.apache.org
For additional commands, e-mail: reviews-help@helix.apache.org