Re: Review Request: KVM initial VPC support

2012-09-04 Thread edison su
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/6883/#review11020 ---

Re: Review Request: KVM initial VPC support

2012-09-04 Thread Marcus Sorensen
On Sept. 4, 2012, 5:06 p.m., edison su wrote: plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/LibvirtComputingResource.java, line 1416 https://reviews.apache.org/r/6883/diff/3/?file=149124#file149124line1416 How about changing the logic to: for (InterfaceDef

Re: Review Request: KVM initial VPC support

2012-09-04 Thread edison su
On Sept. 4, 2012, 5:06 p.m., edison su wrote: plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/LibvirtComputingResource.java, line 1416 https://reviews.apache.org/r/6883/diff/3/?file=149124#file149124line1416 How about changing the logic to: for (InterfaceDef

Re: Review Request: KVM initial VPC support

2012-09-04 Thread edison su
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/6883/#review11028 --- Ship it! Ship It! - edison su On Sept. 3, 2012, 7:37 p.m.,

Re: Review Request: KVM initial VPC support

2012-09-03 Thread Marcus Sorensen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/6883/ --- (Updated Sept. 3, 2012, 7:37 p.m.) Review request for cloudstack. Changes

Review Request: KVM initial VPC support

2012-08-31 Thread Marcus Sorensen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/6883/ --- Review request for cloudstack. Description --- Implements

Re: Review Request: KVM initial VPC support

2012-08-31 Thread edison su
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/6883/#review10962 ---

Re: Review Request: KVM initial VPC support

2012-08-31 Thread Marcus Sorensen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/6883/ --- (Updated Sept. 1, 2012, 5:11 a.m.) Review request for cloudstack. Changes