[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

Re: [libvirt] [PATCH] Fix failing virGetHostname.



On 05/21/2010 12:27 PM, Cole Robinson wrote:
> Hmm, why isn't one of the fallback options:
> 
> if (conn)
>     hostname = parse_uri(conn.get_uri())
>     if hostname != "localhost":
> 	return hostname
> 
> Seems like if MigratePrepare2 dconn is the remote connection, we are
> guaranteed to have a resolvable hostname in the URI (well, resolvable to
> the source connection at least).

(sorry I was late in replying to this)

The thing is, I don't think that is generally going to be useful.  At the
moment, all of the callers pass NULL as conn.  Even if we were to make
some of the callers pass a real conn, because of the way the remote driver
works it doesn't really map back to the IP address you care about.  That is,
at the place where you are running virsh (for instance), conn.uri() is actually
something useful.  But by the time you've remoted through the remote_driver and
made it to the remote libvirtd on the destination of the migration, you are
really doing a "local" connection from libvirtd to the qemu driver, so
conn.uri() basically contains "qemu:///system", which is not particularly
helpful.

-- 
Chris Lalancette


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]