You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@ozone.apache.org by GitBox <gi...@apache.org> on 2021/09/30 20:33:40 UTC
[GitHub] [ozone] errose28 opened a new pull request #2697: HDDS-5645. Chroot S3 requests for a tenant to their corresponding volume.
errose28 opened a new pull request #2697:
URL: https://github.com/apache/ozone/pull/2697
## What changes were proposed in this pull request?
- Direct requests from S3 gateway to corresponding Ozone volume based on the tenant corresponding to the given access ID.
- Refactor usages of access ID/user principal/group principal to better reflect which is required for each method call.
## What is the link to the Apache JIRA
HDDS-5645
## How was this patch tested?
- Integration test for object store added.
- Manual testing with s3 gateway
## TODO
- [ ] Determine best way to retrieve access ID from s3 gateway.
- Without this implemented e2e s3 tests do not work.
- [ ] More thorough documentation of changes.
--
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
[GitHub] [ozone] errose28 commented on pull request #2697: HDDS-5645. Chroot S3 requests for a tenant to their corresponding volume.
Posted by GitBox <gi...@apache.org>.
errose28 commented on pull request #2697:
URL: https://github.com/apache/ozone/pull/2697#issuecomment-941750074
All tests s3gateway module passing locally now.
--
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
[GitHub] [ozone] avijayanhwx commented on pull request #2697: HDDS-5645. Chroot S3 requests for a tenant to their corresponding volume.
Posted by GitBox <gi...@apache.org>.
avijayanhwx commented on pull request #2697:
URL: https://github.com/apache/ozone/pull/2697#issuecomment-941235268
@errose28 Can you take a look at the failing Unit tests? The failures seem related.
--
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
[GitHub] [ozone] errose28 merged pull request #2697: HDDS-5645. Chroot S3 requests for a tenant to their corresponding volume.
Posted by GitBox <gi...@apache.org>.
errose28 merged pull request #2697:
URL: https://github.com/apache/ozone/pull/2697
--
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
[GitHub] [ozone] errose28 commented on pull request #2697: HDDS-5645. Chroot S3 requests for a tenant to their corresponding volume.
Posted by GitBox <gi...@apache.org>.
errose28 commented on pull request #2697:
URL: https://github.com/apache/ozone/pull/2697#issuecomment-941241829
--
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
[GitHub] [ozone] errose28 commented on pull request #2697: HDDS-5645. Chroot S3 requests for a tenant to their corresponding volume.
Posted by GitBox <gi...@apache.org>.
errose28 commented on pull request #2697:
URL: https://github.com/apache/ozone/pull/2697#issuecomment-941241829
Yes failing cases need some mocks updated since I changed the internals a bit. Will get them fixed soon.
--
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
[GitHub] [ozone] avijayanhwx commented on pull request #2697: HDDS-5645. Chroot S3 requests for a tenant to their corresponding volume.
Posted by GitBox <gi...@apache.org>.
avijayanhwx commented on pull request #2697:
URL: https://github.com/apache/ozone/pull/2697#issuecomment-941235268
@errose28 Can you take a look at the failing Unit tests? The failures seem related.
--
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
[GitHub] [ozone] errose28 merged pull request #2697: HDDS-5645. Chroot S3 requests for a tenant to their corresponding volume.
Posted by GitBox <gi...@apache.org>.
errose28 merged pull request #2697:
URL: https://github.com/apache/ozone/pull/2697
--
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
[GitHub] [ozone] errose28 commented on pull request #2697: HDDS-5645. Chroot S3 requests for a tenant to their corresponding volume.
Posted by GitBox <gi...@apache.org>.
errose28 commented on pull request #2697:
URL: https://github.com/apache/ozone/pull/2697#issuecomment-942588450
Thanks for reviews @avijayanhwx @prashantpogde
--
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