NickJavaDev88 commented on PR #10888:
URL: https://github.com/apache/ozone/pull/10888#issuecomment-5113044085

   Thanks for the review, @sreejasahithi! And thanks to @chungen0126 as well,   
                                                                                
                     
     who pointed me toward posting the fork's CI result here and adding a robot 
                                                                                
                       
     test case for this scenario.                                               
                                                                                
                       
                                                                                
                                                                                
                       
     - Updated `AbstractS3SDKV1Tests#testS3LifecycleConfigurationDelete` to 
expect                                                                          
                           
       the idempotent behavior (no exception) instead of 
`AmazonServiceException`                                                        
                                              
       when deleting a lifecycle configuration that never existed.              
                                                                                
                       
     - Added a robot test case (`Delete bucket non-lifecycle configuration` in  
                                                                                
                       
       `bucketlifecycle.robot`) covering delete on a bucket that never had a    
                                                                                
                       
       lifecycle configuration, alongside the existing case that only covers    
                                                                                
                       
       delete after put.                                                        
                                                                                
                       
     - CI on my fork: 
https://github.com/NickJavaDev88/ozone/actions/runs/30421344528                 
                                                                                
 
                                                                                
                                                                                
                       
     Let me know if anything else needs adjusting.


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