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

   ## What changes were proposed in this pull request?
   
   Share cluster among integration tests with ACL enabled and native authorizer:
   * `TestBucketOwner`
   * `TestOzoneManagerListVolumes`
   * `TestRecursiveAclWithFSO`
   
   `TestOzoneManagerListVolumes` has some test cases with ACL disabled.  So 
this test class is added to both `AclTests` and `NonHATests`, to execute with 
ACL enabled and disabled, respectively.  Use assumptions to skip tests in 
specific environments, instead of restarting OM with the desired config.
   
   Update `TestOzoneManagerListVolumes` to dynamically reconfigure 
`allowListAllVolumes` instead of starting new cluster.
   
   https://issues.apache.org/jira/browse/HDDS-12245
   
   ## How was this patch tested?
   
   Before:
   
   ```
   [INFO] Tests run: 3, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 
37.001 s - in org.apache.hadoop.ozone.om.TestBucketOwner
   [INFO] Tests run: 2, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 
67.618 s - in org.apache.hadoop.ozone.om.TestRecursiveAclWithFSO
   [INFO] Tests run: 6, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 
41.184 s - in org.apache.hadoop.ozone.om.TestOzoneManagerListVolumes
   ```
   
   After:
   
   ```
   [INFO] Running org.apache.ozone.test.TestOzoneNonHAWithNativeACL
   [INFO] Tests run: 2, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 1.335 
s - in org.apache.ozone.test.AclTests$RecursiveAclWithFSO
   [WARNING] Tests run: 8, Failures: 0, Errors: 0, Skipped: 4, Time elapsed: 
0.79 s - in org.apache.ozone.test.AclTests$OzoneManagerListVolumes
   [INFO] Tests run: 3, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.514 
s - in org.apache.ozone.test.AclTests$BucketOwner
   [INFO] Tests run: 0, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 
37.824 s - in org.apache.ozone.test.TestOzoneNonHAWithNativeACL
   
   ...
   
   [WARNING] Tests run: 8, Failures: 0, Errors: 0, Skipped: 4, Time elapsed: 
1.078 s - in org.apache.ozone.test.NonHATests$OzoneManagerListVolumes
   ```
   
   CI:
   https://github.com/adoroszlai/ozone/actions/runs/13219326230


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