Change dynamic loading

If we use assignment method (#method=) we don't get return value, so we
never know that manager fails to load.
Also we do not want to rescue failures of input/output/source loading,
they are catastrophic and should fail whole process, loading of model is
not catastrophic.

1 files changed, 5 insertions, 5 deletions

diff --git a/lib/oxidized/nodes.rb b/lib/oxidized/nodes.rb
index 9a3d3bd..0d9fb47 100644
--- a/lib/oxidized/nodes.rb
+++ b/lib/oxidized/nodes.rb
@@ -9,14 +9,14 @@ module Oxidized
       with_lock do
         new = []
         @source = CFG.source[:default]
-        Oxidized.mgr.source = @source
+        Oxidized.mgr.add_source @source
         Oxidized.mgr.source[@source].new.load.each do |node|
           begin
-            n = Node.new node
-          rescue LoadError => err
-            Log.warn 'load error %s with node %s' % [err.message, node]
+            _node = Node.new node
+            new.push _node
+          rescue ModelNotFound => err
+            Log.error "node %s raised %s with message %s" % [node, err.class, err.message]
           end
-          new.push n if n
         end
         replace new
       end