I saw it a bit late... But I just found it need to depends on agent.properties: network.bridge.type=openvswitch
How would that happen if user didn't add host before? Maybe we would just try "ovs-vsctl show" to see if we can find bridge for ovs or not, then overwrite the network.bridge.type? --Sheng On Mon, May 27, 2013 at 12:30 AM, ASF Subversion and Git Services < asf...@urd.zones.apache.org> wrote: > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/11428/ > > Commit a16b707250fde01f24c9ab6f11189b630dd70962 in branch refs/heads/master > from Hiroaki KAWAI <ka...@stratosphere.co.jp> > [ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=a16b707 ] > > CLOUDSTACK-2327: make cloud-setup-agent ovs aware > > Ovs brcompat will be obsolete, so if network.bridge.type was > set to openvswitch, we'll use ovs command explicitly. > > Signed-off-by: Hiroaki KAWAI <ka...@stratosphere.co.jp> > > > - ASF > > On May 27th, 2013, 1:46 a.m., Hiroaki Kawai wrote: > Review request for cloudstack, Sheng Yang and Hugo Trippaers. > By Hiroaki Kawai. > > *Updated May 27, 2013, 1:46 a.m.* > Description > > Ovs brcompat will be obsolete, so if network.bridge.type was > set to openvswitch, we'll use ovs command explicitly. > > Testing > > Tested with my local machines. > - create .rpm, .deb > - install it (CentOS 6.4 and Ubuntu 13.04) > - add the computing node in management server > > *Bugs: * CLOUDSTACK-2327 > Diffs > > - agent/bindir/cloud-setup-agent.in (5e2ba09) > - python/lib/cloudutils/globalEnv.py (94d981f) > - python/lib/cloudutils/networkConfig.py (b6b729a) > - python/lib/cloudutils/serviceConfig.py (1e32d0f) > > View Diff <https://reviews.apache.org/r/11428/diff/> >
