allow setting IP from source

sql/csv can now set IP as well as name, useful if you don't have FQDN. If IP is given, then we don't try to resolve name. closes #78
ytti · Feb 21, 2015 · 61c128c · 61c128c
1 parent 1a9a814
commit 61c128c
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,4 +1,5 @@
 # 0.3.1
+- FEATURE: allow setting IP address in addition to name in source (SQL/CSV)
 - BUGFIX: xos while using telnet (by @fhibler)
 - BUGFIX: ironware logout on some models (by @fhibler)
 

diff --git a/lib/oxidized/node.rb b/lib/oxidized/node.rb
@@ -10,7 +10,8 @@ class Node
     alias :running? :running
     def initialize opt
       @name           = opt[:name]
-      @ip             = Resolv.getaddress @name
+      @ip             = IPAddr.new(opt[:ip]).to_s rescue nil
+      @ip           ||= Resolv.getaddress @name
       @group          = opt[:group]
       @input          = resolve_input opt
       @output         = resolve_output opt