pakapoj-tul opened a new pull request #2343: URL: https://github.com/apache/ozone/pull/2343
HDDS-5193. change to use getShortUserName instead of getUsername ## What changes were proposed in this pull request? IMHO, in kerberized cluster identity `[email protected]` and `pakapoj_tul` are equal so I propose to use `getShortUsername()` instead of `getShortUsername()` from UGI interface when assign and/or compare any ACLs. To leverage **auth_to_local** property to translate any identity from `auth:KERBEROS` to plain username which should be consistent with identity from `auth:TOKEN` ## What is the link to the Apache JIRA https://issues.apache.org/jira/browse/HDDS-5193 Please replace this section with the link to the Apache JIRA) ## How was this patch tested? I manually tested on my ozone cluster, bare metal deployment. there are following tests ``` ozone sh vol create /vol1 ozone sh bucket create /vol1/bucket1 ozone fs -put sample_seq_data.snappy.orc /vol1/bucket1 ozone sh vol addacl --acl=user:pakapoj_tul:a /vol1 ozone sh bucket addacl --acl=user:pakapoj_tul:a /vol1/bucket1 ozone sh key addacl --acl=user:pakapoj_tul:a /vol1/bucket1/sample_seq_data.snappy.orc ozone sh token get -t ozone.token ozone s3 getsecret --om-service-id=dev-ozone aws configure aws s3 ls --endpoint http://mtg8-devozonem-01.dev.tap:9878 s3://common-bucket aws s3api --endpoint http://mtg8-devozonem-01.dev.tap:9878 create-bucket --bucket buckettest aws s3api --endpoint http://mtg8-devozonem-01.dev.tap:9878 delete-bucket --bucket buckettest aws s3 cp --endpoint http://mtg8-devozonem-01.dev.tap:9878 README.md s3://common-bucket aws s3 rm --endpoint http://mtg8-devozonem-01.dev.tap:9878 s3://common-bucket/README.md ``` also test by running spark application in cluster mode ``` val data = spark.read.format("orc").load(src) data.write.format("orc").save(des) ``` given ``` src = "ofs://dev-ozone/vol1/bucket1/sample_seq_data.snappy.orc" des = "ofs://dev-ozone/vol1/bucket1/mykey" ``` -- 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]
