Merge pull request #347 from rundeck-plugins/RUN-2130

RUN-2130: print an error if the node doesn't have a hostname in generate inventory

src/main/groovy/com/rundeck/plugins/ansible/ansible/AnsibleInventoryBuilder.java

@@ -4,6 +4,7 @@
 import com.dtolabs.rundeck.core.plugins.configuration.ConfigurationException;
 
 import java.io.File;
+import java.io.IOException;
 import java.io.PrintWriter;
 import java.util.Collection;
 import java.util.HashMap;
@@ -25,12 +26,15 @@ public File buildInventory() throws ConfigurationException {
             PrintWriter writer = new PrintWriter(file);
             AnsibleInventory ai = new AnsibleInventory();
             for (INodeEntry e : nodes) {
+                if( e.getHostname() == null ){
+                    throw new ConfigurationException("No hostname for node: " + e.getNodename());
+                }
                 ai.addHost(e.getNodename(), e.getHostname(), new HashMap<String, String>(e.getAttributes()));
             }
             writer.write(new Gson().toJson(ai));
             writer.close();
             return file;
-        } catch (Exception e) {
+        } catch (IOException e) {
             throw new ConfigurationException("Could not write temporary inventory: " + e.getMessage());
         }
     }