Remove the unused shared network validation check as its a bit misleading 52/37652/1
authorSridhar Gaddam <sgaddam@redhat.com>
Fri, 15 Apr 2016 09:36:33 +0000 (15:06 +0530)
committerSridhar Gaddam <sgaddam@redhat.com>
Fri, 15 Apr 2016 09:38:30 +0000 (15:08 +0530)
commita894617879f2c2f8ea7b70355b9721a855cc635a
tree9628b60d9c0ff2a7d65fefdb217154071cab143c
parent3997f73ee6cd22c17de329a1ac4113e265b06203
Remove the unused shared network validation check as its a bit misleading

ODL Neutron northbound is now MDSAL based (from Beryllium).
The canCreateNetwork/canUpdateNetwork checks are no longer
invoked in NetworkHandler of netvirt.

Observations with stable/Beryllium and master branch.
-----------------------------------------------------
In stable/beryllium and master branches an admin is able
to create a shared network. Two tenants are able to spawn
VMs (on the shared network), which successfully acquire
dhcp-ip and are able to communicate with the VM belonging
to the other tenant. Verified this use-case on a multi-node
setup.

It appears like shared networks are indeed supported. So, we
can remove this un-used validation as its misleading.

Observations with stable/Lithium SR4 release.
---------------------------------------------
An admin is unable to create a shared tenant network but shared
external networks are supported. Before removing the validation
for Lithium branch we have to see if there was any specific
reason why the check was inplace.

Change-Id: I62ab980110c58f07a9e161431b64a64590c1e410
Signed-off-by: Sridhar Gaddam <sgaddam@redhat.com>
openstack/net-virt/src/main/java/org/opendaylight/netvirt/openstack/netvirt/NetworkHandler.java
openstack/net-virt/src/test/java/org/opendaylight/netvirt/openstack/netvirt/NetworkHandlerTest.java