diff options
author | Saku Ytti <saku@ytti.fi> | 2014-03-17 11:40:26 +0200 |
---|---|---|
committer | Saku Ytti <saku@ytti.fi> | 2014-03-17 11:40:26 +0200 |
commit | bb2e48ba7e9cc76b987a11952ce7585105911ace (patch) | |
tree | 7419487356b4df9a8962e867c39fe3679453b3c8 /lib/oxidized/model | |
parent | 9599347820c75aedda886914cca54cec45f48c63 (diff) |
Fix JunOS model
Block must return string, it returned nil which broke it.
Also coerce type to string in future, so block won't fail so
catastrophically, just output missing.
Diffstat (limited to 'lib/oxidized/model')
-rw-r--r-- | lib/oxidized/model/junos.rb | 4 | ||||
-rw-r--r-- | lib/oxidized/model/model.rb | 4 |
2 files changed, 5 insertions, 3 deletions
diff --git a/lib/oxidized/model/junos.rb b/lib/oxidized/model/junos.rb index 4bca998..5b3d11b 100644 --- a/lib/oxidized/model/junos.rb +++ b/lib/oxidized/model/junos.rb @@ -20,10 +20,12 @@ class JunOS < Oxidized::Model end post do + out = '' case @model when 'mx960' - cmd('show chassis fabric reachability') { |cfg| comment cfg } + out << cmd('show chassis fabric reachability') { |cfg| comment cfg } end + out end cmd 'show chassis hardware' do |cfg| diff --git a/lib/oxidized/model/model.rb b/lib/oxidized/model/model.rb index 70b307f..ccc03b4 100644 --- a/lib/oxidized/model/model.rb +++ b/lib/oxidized/model/model.rb @@ -125,10 +125,10 @@ module Oxidized data << out.to_s end procs[:pre].each do |pre_proc| - pre << instance_eval(&pre_proc) + pre << instance_eval(&pre_proc).to_s end procs[:post].each do |post_proc| - data << instance_eval(&post_proc) + data << instance_eval(&post_proc).to_s end pre + data end |