same8891 opened a new pull request, #1792:
URL: https://github.com/apache/karaf/pull/1792

   ### Description
   
   #### Test failure Reproduction
   We use [idflakies](https://github.com/UT-SE-Research/iDFlakies) to detect 
the flaky test.
   ```
   mvn install -pl main -am -DskipTests
   mvn -pl main idflakies:detect -Ddetector.detector_type=random-class-method 
-Ddt.randomize.rounds=10 -Ddt.detector.original_order.all_must_pass=false
   
   ```
   #### Root cause and fix
   This error is caused by line 140. 
   ```
   Assert.assertEquals(100, sl.getStartLevel()); 
   ```
   It asserted the startlevel wirh 100 which means the process got the lock.  
The flaky is caused by the test didn't wait for the process to change the start 
level. So, sometimes the test will fail by the start level is 1 which means the 
process didn't have the lock. To fix this, I added
   ```
   Thread.sleep(1000);
   ```
   after the process gets the lock to ensure the start level has time to chang.


-- 
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.

To unsubscribe, e-mail: commits-unsubscr...@karaf.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to