trozet / tacker

tacker: specifications a servicevm/device life cycle manager. See https://wiki.openstack.org/wiki/ServiceVM
http://openstack.org
Apache License 2.0
3 stars 7 forks source link

Creating SFC removes other SFCs in ODL #2

Open trozet opened 9 years ago

trozet commented 9 years ago

Not sure where the problem is yet, but should be in the opendaylight.py driver.

trozet commented 9 years ago

Reinaldo informed me its because im using the wrong REST URI + JSON which is overwriting everything. Fixed the correct URIs and testing, which introduces 2 other issues. 1. Need to check that the name of hte SFC is unique (since we use this in path id for ODL). 2. Need to check if SFF already exists in ODL SFC (by IP/port). If it does then we just update the VNFs that are on it, rather than creating another SFF.

trozet commented 9 years ago

Everything except 2 is resolved with this patch: https://github.com/trozet/tacker/commit/cb64d6d160282c18de585c69acb2a285060407e5

Will need to add the logic to do 2 in an upcoming patch. One other issue noticed is creating a new chain with the same VNFs results in removal of the RSPs. Not sure if this is coincidence or related (RSP might be getting eventually deleted since I don't have NSH OVS). Will need to test this after 2 is fixed.