I have a loop where i execute a series of command on a remote machine:
ssh.exec('cd /vmfs/volumes/4c6d95d2-b1923d5d-4dd7-f4ce46baaadc/ghettoVCB; ./ghettoVCB.sh -f vms_to_backup -d dryrun') do|ch, stream, data|
if #{stream} =~ /vmupgrade/
puts value_hosts + " is " + data
puts #{stream}
puts data
end
end
i want to access #{stream} and data outside the do-end loop
I would appreciate any help. Thanks,
Hi Jörg,
I implemented your suggestions, but now i am getting error:
WrapperghettoVCB.rb:49: odd number list for Hash
communicator = {ch: ch, stream: stream, data: data}
^
WrapperghettoVCB.rb:49: syntax error, unexpected ':', expecting '}'
communicator = {ch: ch, stream: stream, data: data}
^
WrapperghettoVCB.rb:49: syntax error, unexpected ':', expecting '='
communicator = {ch: ch, stream: stream, data: data}
^
WrapperghettoVCB.rb:49: syntax error, unexpected ':', expecting '='
communicator = {ch: ch, stream: stream, data: data}
^
WrapperghettoVCB.rb:76: syntax error, unexpected kELSE, expecting kEND
WrapperghettoVCB.rb:80: syntax error, unexpected '}', expecting kEND
You can't. Local variables are local to their scope. That's why they are called local variables.
You could, however, use a variable from an outer scope:
communicator = nil
ssh.exec('...') do |ch, stream, data|
break unless stream =~ /vmupgrade/
puts "#{value_hosts} is #{data}", stream, data
communicator = {ch: ch, stream: stream, data: data}
end
puts communicator
BTW: there were several bugs in your code, that would have prevented it from working anyway regardless of your problem with variable scoping, because you used wrong syntax for dereferencing local variables: the syntax for dereferencing a variable is simply the name of the variable, e.g. foo
, and not #{foo}
(that's simply a syntax error).
Also, there are some other improvements:
do
keyword with a space+
is unusual in Ruby; if you need to concatenate strings, you usually do it with <<
, but more often than not, string interpolation is preferredputs
: if you pass multiple arguments to puts
, it will print all of them on a separate line, you don't need to call it multiple timesIf you love us? You can donate to us via Paypal or buy me a coffee so we can maintain and grow! Thank you!
Donate Us With