commit e23e680fe8e329410126ff4a83ad004ce91b528a
author Jon Maron <jmaron@hortonworks.com> Thu Oct 30 21:31:41 2014 -0400
committer Jon Maron <jmaron@hortonworks.com> Thu Oct 30 21:31:41 2014 -0400
tree 62733bbd7e73938736126b7de517fd1a46dda810
parent 1a071e31dd2f2c4486bcbdfe2f02065efe636545

SLIDER-574 print log message when no service keytabs are localized

slider-core/src/main/java/org/apache/slider/providers/agent/AgentProviderService.java

diff --git a/slider-core/src/main/java/org/apache/slider/providers/agent/AgentProviderService.java b/slider-core/src/main/java/org/apache/slider/providers/agent/AgentProviderService.java
index 934e5f0..4720ced 100644
--- a/slider-core/src/main/java/org/apache/slider/providers/agent/AgentProviderService.java
+++ b/slider-core/src/main/java/org/apache/slider/providers/agent/AgentProviderService.java
@@ -521,9 +521,11 @@
                                                       getClusterName());
       FileStatus[] keytabs = fileSystem.getFileSystem().listStatus(keytabDirPath);
       LocalResource keytabRes;
+      boolean serviceKeytabsDeployed = false;
       for (FileStatus keytab : keytabs) {
         if (!amKeytabName.equals(keytab.getPath().getName())
             && keytab.getPath().getName().endsWith(".keytab")) {
+          serviceKeytabsDeployed = true;
           log.info("Localizing keytab {}", keytab.getPath().getName());
           keytabRes = fileSystem.createAmResource(keytab.getPath(),
             LocalResourceType.FILE);
@@ -532,6 +534,13 @@
                                   keytabRes);
         }
       }
+      if (!serviceKeytabsDeployed) {
+        log.warn("No service keytabs for the application have been localized.  "
+                 + "If the application requires keytabs for secure operation, "
+                 + "please ensure that the required keytabs have been uploaded "
+                 + "to the folder designated by the property {}: {}",
+                 SliderXmlConfKeys.KEY_HDFS_KEYTAB_DIR, keytabDirPath);
+      }
     }
   }