sguggilam commented on a change in pull request #2178:
URL: https://github.com/apache/hadoop/pull/2178#discussion_r463345150
##########
File path:
hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/security/TestUGILoginFromKeytab.java
##########
@@ -100,13 +101,34 @@ public void stopMiniKdc() {
executor.shutdownNow();
}
}
+
+ /**
+ * Login from keytab using the MiniKDC
+ */
+ @Test
+ public void testUGILoginFromKeytab() throws Exception {
+ long beforeLogin = Time.now();
+ String principal = "foo";
+ File keytab = new File(workDir, "foo.keytab");
+ kdc.createPrincipal(keytab, principal);
+
+ UserGroupInformation.loginUserFromKeytab(principal, keytab.getPath());
+ UserGroupInformation ugi = UserGroupInformation.getLoginUser();
+ Assert.assertTrue("UGI should be configured to login from keytab",
+ ugi.isFromKeytab());
+
+ User user = getUser(ugi.getSubject());
+ Assert.assertNotNull(user.getLogin());
+ Assert.assertTrue("User last login time is not set correctly",
Review comment:
Agreed, updated the code
----------------------------------------------------------------
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]
