adoroszlai opened a new pull request, #10429:
URL: https://github.com/apache/ozone/pull/10429

   ## What changes were proposed in this pull request?
   
   Speed up `TestOzoneManagerListVolumesSecure` by reducing KDC and OM restarts.
   
   1. Start KDC and setup principals only once, before all tests.
   2. `ozone.om.volume.listall.allowed` can be set on OM without restart.
   3. Split test cases into two nested classes, depending on 
`ozone.acl.enabled` setting.  (Use `git diff --ignore-space-change` in CLI to 
hide lines only touched by increased indentation.)
   
   https://issues.apache.org/jira/browse/HDDS-15478
   
   ## How was this patch tested?
   
   
[Before](https://github.com/apache/ozone/actions/runs/26932464709/job/79456064367#step:13:6187):
 1 minute
   
   ```
   Tests run: 6, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 59.87 s -- 
in org.apache.hadoop.ozone.om.TestOzoneManagerListVolumesSecure
   ```
   
   
[After](https://github.com/adoroszlai/ozone/actions/runs/26940169171/job/79481224643#step:13:6182):
 ~23 seconds
   
   ```
   Running org.apache.hadoop.ozone.om.TestOzoneManagerListVolumesSecure
   Running 
org.apache.hadoop.ozone.om.TestOzoneManagerListVolumesSecure$AclDisabled
   Tests run: 2, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 12.78 s -- 
in org.apache.hadoop.ozone.om.TestOzoneManagerListVolumesSecure$AclDisabled
   Running 
org.apache.hadoop.ozone.om.TestOzoneManagerListVolumesSecure$AclEnabled
   Tests run: 4, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 8.189 s -- 
in org.apache.hadoop.ozone.om.TestOzoneManagerListVolumesSecure$AclEnabled
   Tests run: 0, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 22.62 s -- 
in org.apache.hadoop.ozone.om.TestOzoneManagerListVolumesSecure
   ```
   
   https://github.com/adoroszlai/ozone/actions/runs/26940169171


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to