search

atlanticwave-sdx / pce

Path Computation Element for AtlanticWave SDX.
https://www.atlanticwave-sdx.net
0 stars 3 forks source link

Adding inter domain links on topology update can result in duplicated links #198

Closed italovalcy closed 1 month ago

italovalcy commented 1 month ago

Hi,

The current implementation of inter-domain links computation when updating the topology can result in duplicated links:

$ curl -s http://0.0.0.0:8080/SDX-Controller/1.0.0/topology | jq -r '.links[].id'
urn:sdx:link:interdomain:ampath.net:Ampath1:40:sax.net:Sax01:40
urn:sdx:link:interdomain:ampath.net:Ampath2:40:sax.net:Sax02:40
urn:sdx:link:tenet.ac.za:Tenet01/1_Tenet02/1
urn:sdx:link:tenet.ac.za:Tenet01/2_Tenet03/2
urn:sdx:link:interdomain:sax.net:Sax02:41:tenet.ac.za:Tenet02:41
urn:sdx:link:interdomain:sax.net:Sax01:41:tenet.ac.za:Tenet01:41
urn:sdx:link:interdomain:ampath.net:Ampath1:40:sax.net:Sax01:40
urn:sdx:link:interdomain:ampath.net:Ampath2:40:sax.net:Sax02:40
urn:sdx:link:interdomain:ampath.net:Ampath1:40:sax.net:Sax01:40
urn:sdx:link:interdomain:ampath.net:Ampath2:40:sax.net:Sax02:40
urn:sdx:link:interdomain:ampath.net:Ampath1:40:sax.net:Sax01:40
urn:sdx:link:interdomain:ampath.net:Ampath2:40:sax.net:Sax02:40
urn:sdx:link:interdomain:sax.net:Sax01:41:tenet.ac.za:Tenet01:41
urn:sdx:link:interdomain:ampath.net:Ampath1:40:sax.net:Sax01:40
urn:sdx:link:interdomain:sax.net:Sax02:41:tenet.ac.za:Tenet02:41
urn:sdx:link:interdomain:ampath.net:Ampath2:40:sax.net:Sax02:40
urn:sdx:link:interdomain:ampath.net:Ampath1:40:sax.net:Sax01:40
urn:sdx:link:interdomain:ampath.net:Ampath2:40:sax.net:Sax02:40
urn:sdx:link:interdomain:ampath.net:Ampath1:40:sax.net:Sax01:40
urn:sdx:link:interdomain:ampath.net:Ampath2:40:sax.net:Sax02:40
urn:sdx:link:interdomain:ampath.net:Ampath1:40:sax.net:Sax01:40
urn:sdx:link:interdomain:ampath.net:Ampath2:40:sax.net:Sax02:40
urn:sdx:link:interdomain:sax.net:Sax01:41:tenet.ac.za:Tenet01:41
urn:sdx:link:interdomain:ampath.net:Ampath1:40:sax.net:Sax01:40
urn:sdx:link:interdomain:sax.net:Sax02:41:tenet.ac.za:Tenet02:41
urn:sdx:link:interdomain:ampath.net:Ampath2:40:sax.net:Sax02:40
urn:sdx:link:sax.net:Sax01/1_Sax02/1
urn:sdx:link:interdomain:sax.net:Sax01:41:tenet.ac.za:Tenet01:41
urn:sdx:link:interdomain:ampath.net:Ampath1:40:sax.net:Sax01:40
urn:sdx:link:interdomain:sax.net:Sax02:41:tenet.ac.za:Tenet02:41
urn:sdx:link:interdomain:ampath.net:Ampath2:40:sax.net:Sax02:40
urn:sdx:link:ampath.net:Ampath1/2_Ampath3/2
urn:sdx:link:ampath.net:Ampath2/3_Ampath3/3
urn:sdx:link:ampath.net:Ampath1/1_Ampath2/1
urn:sdx:link:interdomain:ampath.net:Ampath1:40:sax.net:Sax01:40
urn:sdx:link:interdomain:ampath.net:Ampath2:40:sax.net:Sax02:40