Michael can u reply on it?

From: Ohad Levy [mailto:notifications@github.com]
Sent: Wednesday, April 02, 2014 9:37 AM
To: theforeman/foreman
Cc: Operations
Subject: Re: [foreman] Xenserver support (#1345)


In lib/foreman/model/xenserver.rb:

> +      raise "No console fore vm #{vm.name}" if console == nil

> +

> +      session_ref = (vm.service.instance_variable_get :@connection).instance_variable_get :@credentials

> +      fullURL = "#{console.location}&session_id=#{session_ref}"

> +      tunnel = VNCTunnel.new fullURL

> +      tunnel.start

> +      logger.info "VNCTunnel started"

> +      WsProxy.start(:host => tunnel.host, :host_port => tunnel.port, :password => '').merge(:type => 'vnc', :name=> vm.name)

> +

> +    rescue Error => e

> +      logger.warn e

> +      raise e

> +    end

> +

> +    def hypervisor

> +      client.hosts.first

does it really make sense to use only the first hypervisor?

—
Reply to this email directly or view it on GitHub<https://github.com/theforeman/foreman/pull/1345/files#r11193928>.