ThomasMarquardt opened a new pull request #2504:
URL: https://github.com/apache/hadoop/pull/2504
DETAILS:
The previous commit for HADOOP-17397 was not the correct fix.
DelegationSASGenerator.getDelegationSAS
should return sp=p for the set-permission and set-acl operations. The
tests have also been updated as
follows:
1. When saoid and suoid are not specified, skoid must have an RBAC role
assignment which grants
Microsoft.Storage/storageAccounts/blobServices/containers/blobs/modifyPermissions/action
and sp=p
to set permissions or set ACL.
2. When saoid or suiod is specified, same as 1) but furthermore the
saoid or suoid must be an owner of
the file or directory in order for the operation to succeed.
3. When saoid or suiod is specified, the ownership check is bypassed by
also including 'o' (ownership)
in the SAS permission (for example, sp=op). Note that 'o' grants the
saoid or suoid the ability to
change the file or directory owner to themself, and they can also
change the owning group. Generally
speaking, if a trusted authorizer would like to give a user the
ability to change the permissions or
ACL, then that user should be the file or directory owner.
TEST RESULTS:
namespace.enabled=true
auth.type=SharedKey
-------------------
$mvn -T 1C -Dparallel-tests=abfs -Dscale -DtestsThreadCount=8 clean
verify
Tests run: 90, Failures: 0, Errors: 0, Skipped: 0
Tests run: 462, Failures: 0, Errors: 0, Skipped: 24
Tests run: 208, Failures: 0, Errors: 0, Skipped: 24
namespace.enabled=true
auth.type=OAuth
-------------------
$mvn -T 1C -Dparallel-tests=abfs -Dscale -DtestsThreadCount=8 clean
verify
Tests run: 90, Failures: 0, Errors: 0, Skipped: 0
Tests run: 462, Failures: 0, Errors: 0, Skipped: 70
Tests run: 208, Failures: 0, Errors: 0, Skipped: 141
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
