Re: VNC listen address for KVM

2013-03-22 Thread Chip Childers
21, 2013 5:19 PM > > To: dev@cloudstack.apache.org > > Cc: cloudstack-...@incubator.apache.org > > Subject: Re: VNC listen address for KVM > > > > On Thu, Mar 21, 2013 at 04:11:56PM -0700, Edison Su wrote: > > > At least, need to revert the following change: > >

RE: VNC listen address for KVM

2013-03-21 Thread Edison Su
I'll create a patch for it. > -Original Message- > From: Chip Childers [mailto:chip.child...@sungard.com] > Sent: Thursday, March 21, 2013 5:19 PM > To: dev@cloudstack.apache.org > Cc: cloudstack-...@incubator.apache.org > Subject: Re: VNC listen address for KVM &g

Re: VNC listen address for KVM

2013-03-21 Thread Chip Childers
gt; -Original Message- > > From: Marcus Sorensen [mailto:shadow...@gmail.com] > > Sent: Thursday, March 21, 2013 2:02 PM > > To: cloudstack-...@incubator.apache.org > > Subject: Re: VNC listen address for KVM > > > > To be clear, it doesn't break conso

RE: VNC listen address for KVM

2013-03-21 Thread Edison Su
null); in LibvirtComputingResource in 4.1, otherwise, live migration will not work as Wido pointed out. > -Original Message- > From: Marcus Sorensen [mailto:shadow...@gmail.com] > Sent: Thursday, March 21, 2013 2:02 PM > To: cloudstack-...@incubator.apache.org > Subject: Re: VNC listen a

Re: VNC listen address for KVM

2013-03-21 Thread Marcus Sorensen
To be clear, it doesn't break consoleproxy for me, but it breaks migration as wido mentions in his long first post. I think he intended to fix it with the new libvirt 0.5.0 bindings rather than roll back, but it didn't happen before the cut. It should probably be rolled back on both 4.1 and master

Re: VNC listen address for KVM

2013-03-21 Thread Marcus Sorensen
The change was in server side code, so maybe. On Jan 4, 2013 1:09 PM, "Wido den Hollander" wrote: > Hi, > > I just noticed that CLOUDSTACK-411 got resolved which is related to > CLOUDSTACK-410 > > * > https://issues.apache.org/**jira/browse/CLOUDSTACK-410

Re: VNC listen address for KVM

2013-03-21 Thread Chip Childers
On Thu, Mar 21, 2013 at 02:27:48PM -0600, Marcus Sorensen wrote: > This is breaking 4.1. Should we revert the commit for this release? Perhaps. Do you think it's related to CLOUDSTACK-1740 though? I'd assume that this is a KVM change being discussed below, but 1740 is for Xen. Perhaps it's a co

Re: VNC listen address for KVM

2013-03-21 Thread Marcus Sorensen
This is breaking 4.1. Should we revert the commit for this release? On Sat, Jan 5, 2013 at 10:29 AM, John Kinsella wrote: > nice, hadn't considered ACLing that at the hypervisor level > > On Jan 4, 2013, at 12:09 PM, Wido den Hollander wrote: > >> Hi, >> >> I just noticed that CLOUDSTACK-411 got