YARN-10247. Application priority queue ACLs are not respected. Contributed by Sunil G
This commit is contained in:
parent
db6252b6c3
commit
410c605aec
@ -2686,7 +2686,7 @@ public Priority checkAndGetApplicationPriority(
|
|||||||
}
|
}
|
||||||
|
|
||||||
// Lets check for ACLs here.
|
// Lets check for ACLs here.
|
||||||
if (!appPriorityACLManager.checkAccess(user, queuePath, appPriority)) {
|
if (!appPriorityACLManager.checkAccess(user, normalizeQueueName(queuePath), appPriority)) {
|
||||||
throw new YarnException(new AccessControlException(
|
throw new YarnException(new AccessControlException(
|
||||||
"User " + user + " does not have permission to submit/update "
|
"User " + user + " does not have permission to submit/update "
|
||||||
+ applicationId + " for " + appPriority));
|
+ applicationId + " for " + appPriority));
|
||||||
|
@ -143,6 +143,7 @@ private void submitAppToRMWithInValidAcl(String submitter,
|
|||||||
.newInstance(appSubmissionContext);
|
.newInstance(appSubmissionContext);
|
||||||
try {
|
try {
|
||||||
submitterClient.submitApplication(submitRequest);
|
submitterClient.submitApplication(submitRequest);
|
||||||
|
Assert.fail();
|
||||||
} catch (YarnException ex) {
|
} catch (YarnException ex) {
|
||||||
Assert.assertTrue(ex.getCause() instanceof RemoteException);
|
Assert.assertTrue(ex.getCause() instanceof RemoteException);
|
||||||
}
|
}
|
||||||
|
Loading…
Reference in New Issue
Block a user