On Wed, Jan 25, 2012 at 8:36 PM, Michal Fojtik <[email protected]> wrote:
> ACK.
>
> I would rather prefer ((xml/'guest_info//ip') but this seems to work.
>

but ips is a group of ip...it's meaningful :)

Francesco

>  -- Michal
>
> --
> Michal Fojtik
> Red Hat Inc.
>
>
> On Wednesday, January 25, 2012 at 8:17 PM, Francesco Vollero wrote:
>
>>
>> Signed-off-by: Francesco Vollero <[email protected] 
>> (mailto:[email protected])>
>> ---
>> .../lib/deltacloud/drivers/rhevm/rhevm_client.rb | 2 +-
>> 1 files changed, 1 insertions(+), 1 deletions(-)
>>
>> diff --git a/server/lib/deltacloud/drivers/rhevm/rhevm_client.rb 
>> b/server/lib/deltacloud/drivers/rhevm/rhevm_client.rb
>> index 26b15b7..5a75143 100644
>> --- a/server/lib/deltacloud/drivers/rhevm/rhevm_client.rb
>> +++ b/server/lib/deltacloud/drivers/rhevm/rhevm_client.rb
>> @@ -373,7 +373,7 @@ module RHEVM
>> @storage = ((xml/'disks/disk/size').first.text rescue nil)
>> @macs = (xml/'nics/nic/mac').collect { |mac| mac[:address] }
>> @creation_time = (xml/'creation_time').text
>> - @ip = ((xml/'guest_info/ip').first[:address] rescue nil)
>> + @ip = ((xml/'guest_info/ips/ip').first[:address] rescue nil)
>> @vnc = {
>> :address => ((xml/'display/address').first.text rescue "127.0.0.1"),
>> :port => ((xml/'display/port').first.text rescue "5890")
>> --
>> 1.7.7.6
>
>
>

Reply via email to