[GENIUS-204] TEP is part of old and new TZ when tz is updated after del-manager Created: 06/Aug/18  Updated: 06/Aug/18

Status: In Review
Project: genius
Component/s: ITM
Affects Version/s: None
Fix Version/s: None

Type: Bug Priority: Medium
Reporter: Tarun Thakur Assignee: Tarun Thakur
Resolution: Unresolved Votes: 0
Labels: None
Remaining Estimate: 0 minutes
Time Spent: 1 day
Original Estimate: Not Specified


 Description   

TEP is part of two zones i.e. old transport zone and new transport zone when zone is removed after del-manager.

Ideally, it should get removed from old TZ when manager is connected back and new TZ is added from NB.

Steps:
1. Make sure setup is up with TZB transport-zone and tunnels
2. now execute "ovs-vsctl del-manager" in any of the ovs.
3. remove transport zone TZB from ovsdb and create new TZC
ovs-vsctl remove O . external_ids transport-zone
ovs-vsctl set O . external_ids:transport-zone=TZC
4. connect back manager to ODL.
sudo ovs-vsctl set-manager tcp:192.168.40.28:6640
5. check /restconf/operational/itm:not-hosted-transport-zones/ and create a TZC from NB
curl -X POST -H "Content-Type:application/json" -d @itm_tzc http://localhost:8181/restconf/config/itm:transport-zones

 Comments   
Comment by Tarun Thakur [ 06/Aug/18 ]
  • Fail scenario
  1. TZB was present and TEPs were part of it.
  2. ovsdb connection is removed by running delete-manager command on switch
  3. Change transport-zone of TEPs from TZB to TZC
  4. ovsdb connection is set again by running set-manager command on switch
  5. TEPs go under teps-not-hosted since TZC is not yet present. It is noted here that TEPs are still part of TZB in ITM config DS.
  6. TZC is created from northbound
  7. TEPs are moved from teps-not-hosted to TZC.
  8. In the end, TEPs are part of TZB and TZC both. This is issue.
  • Fix provided

When TEPs add notification come again with updated TZC, then check if same TEP (key: dpid) is already present in any other TZ. If present, then remove TEP from old TZ and then add TEP into newly configured TZ.

Comment by Faseela K [ 06/Aug/18 ]

Tarun, let us close the TR, once the patches are merged

Comment by Tarun Thakur [ 06/Aug/18 ]

Ok, understood. Thanks for updating the bug.

Generated at Wed Feb 07 20:00:10 UTC 2024 using Jira 8.20.10#820010-sha1:ace47f9899e9ee25d7157d59aa17ab06aee30d3d.