[GitHub] [ozone] GeorgeJahad commented on pull request #4291: HDDS 6986. Update ozone ranger plugin to handle snapshots

["GeorgeJahad (via GitHub)" <gi...@apache.org>]

GeorgeJahad commented on PR #4291:
URL: https://github.com/apache/ozone/pull/4291#issuecomment-1438919466

   > Since the change is mostly done in RequestContext, it should be easy to add support for Ozone native ACL in OzoneNativeAuthorizer as well.
   
   FYI @smengcl  I believe the OzoneNativeAuthorizer in the master branch already works with snapshots.  When I created the OmMetadataReader, I tried to implement it in such a way that ACL's would be read from the snapshot assigned to the OmMetadataReader.
   
   I never wrote the tests for it because @prashantpogde asked me to put that work on hold.
   
   But those tests are now being written by my team mate: @mladjan-gadzic 
   
   They appear to be all working, and he will submit the PR in the next few days.
   
   Bottom line, I don't believe there are any more changes needed for the native authorizor, just some tests which will be submitted shortly.
   
   


-- 
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: issues-unsubscribe@ozone.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@ozone.apache.org
For additional commands, e-mail: issues-help@ozone.apache.org