cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Sheng Yang" <sh...@yasker.org>
Subject Re: Review Request 12972: Update CitrixResourceBase to use new vmdata programming method
Date Wed, 31 Jul 2013 03:22:29 GMT


> On July 31, 2013, 3:22 a.m., Sheng Yang wrote:
> > Ship It!

Looks fine to me.


- Sheng


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/12972/#review24316
-----------------------------------------------------------


On July 31, 2013, 2:52 a.m., Marcus Sorensen wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/12972/
> -----------------------------------------------------------
> 
> (Updated July 31, 2013, 2:52 a.m.)
> 
> 
> Review request for cloudstack, Prasanna Santhanam and Sheng Yang.
> 
> 
> Bugs: CLOUDSTACK-3163
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> -------
> 
> Update CitrixResourceBase to use new vmdata programming method, removing dozens of ssh/scp
calls on router and vm startup. This will get rid of vm_data.sh.
> 
> 
> Diffs
> -----
> 
>   core/src/com/cloud/agent/resource/virtualnetwork/VirtualRoutingResource.java e5f922d

>   patches/systemvm/debian/config/opt/cloud/bin/vmdata.py PRE-CREATION 
>   patches/systemvm/debian/config/opt/cloud/bin/vmdata_kvm.py f508032 
>   plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/CitrixResourceBase.java
fa7f2ec 
>   scripts/network/domr/vm_data.sh c861723 
>   scripts/vm/hypervisor/xenserver/vmops ff33c2d 
>   scripts/vm/hypervisor/xenserver/xcposs/patch 4d07c76 
>   scripts/vm/hypervisor/xenserver/xcposs/vmops 52625e1 
>   scripts/vm/hypervisor/xenserver/xcpserver/patch 7e92d5a 
>   scripts/vm/hypervisor/xenserver/xenserver56/patch 8abd6b2 
>   scripts/vm/hypervisor/xenserver/xenserver56fp1/patch 901f6de 
>   scripts/vm/hypervisor/xenserver/xenserver60/patch d7da374 
> 
> Diff: https://reviews.apache.org/r/12972/diff/
> 
> 
> Testing
> -------
> 
> None, please test.
> 
> 
> Thanks,
> 
> Marcus Sorensen
> 
>


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message