----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12775/#review23617 -----------------------------------------------------------
Review 12775 PASSED the build test The url of build cloudstack-master-with-patch #6 is : http://jenkins.cloudstack.org/job/cloudstack-master-with-patch/6/ - Jenkins Cloudstack.org On July 19, 2013, 9:20 p.m., Marcus Sorensen wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/12775/ > ----------------------------------------------------------- > > (Updated July 19, 2013, 9:20 p.m.) > > > Review request for cloudstack, ilya musayev and Wido den Hollander. > > > Bugs: cloudstack-3163 > > > Repository: cloudstack-git > > > Description > ------- > > Migrate the dozens of ssh calls for KVM VmDataCommand to a single router > command. > > > Diffs > ----- > > > core/src/com/cloud/agent/resource/virtualnetwork/VirtualRoutingResource.java > 0b26220 > patches/systemvm/debian/config/opt/cloud/bin/vmdata_kvm.py PRE-CREATION > scripts/network/domr/vm_data.sh c861723 > > Diff: https://reviews.apache.org/r/12775/diff/ > > > Testing > ------- > > Tested on 4.2 advanced zone, as well as a version against 4.1 (patch applies > with fuzz) since that's what we happen to be using at the moment. Started VM, > then ran 'curl http://10.1.1.1/latest/instance-id', then ran > updateVirtualMachine, setting userdata, and pulled it on the guest via 'curl > http://10.1.1.1/latest/user-data'. > > > Thanks, > > Marcus Sorensen > >
