Marton Greber has posted comments on this change. ( http://gerrit.cloudera.org:8080/23045 )
Change subject: Add REST API integration tests ...................................................................... Patch Set 11: Verified+1 Code-Review+2 The assumption was that calls from REST API are transparently forwarded to the authz logic that already exists. Aside from analysing just the code, we've figured that we should probably do at least "smoke" type of tests in terms of the authz integration tests for REST API. The only thing that came up is the principal name mapping issue, but that has been resolved by Gabi. The thought behind the creation of this patch was along these lines. LGTM, but let's see if Alexey has any more observations. Thanks Alexey for taking a closer look! unrelated test failures: debug: org.apache.kudu.backup.TestKuduBackup tsan: auto_leader_rebalancer-test.0, maintenance_manager-test.0, org.apache.kudu.client.TestKuduClient -- To view, visit http://gerrit.cloudera.org:8080/23045 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ifd3ff0dfd67cbc2b5ed0454372dd2bcea71e2ba3 Gerrit-Change-Number: 23045 Gerrit-PatchSet: 11 Gerrit-Owner: Gabriella Lotz <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Gabriella Lotz <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Marton Greber <[email protected]> Gerrit-Reviewer: Zoltan Chovan <[email protected]> Gerrit-Comment-Date: Mon, 25 Aug 2025 13:06:48 +0000 Gerrit-HasComments: No
