Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Duplicate paths in sciond #2444

Closed
FR4NK-W opened this issue Feb 12, 2019 · 2 comments
Closed

Duplicate paths in sciond #2444

FR4NK-W opened this issue Feb 12, 2019 · 2 comments

Comments

@FR4NK-W
Copy link
Contributor

FR4NK-W commented Feb 12, 2019

Path filtering issue in path combinator:
Multiple identical paths (same interfaces) are returned by sciond.

How to reproduce:
Used topology: modified Tiny.topo, add an intermediate AS with redundant links.

diff --git a/topology/Tiny.topo b/topology/Tiny.topo
index 8b564b1..498959c 100644
--- a/topology/Tiny.topo
+++ b/topology/Tiny.topo
@@ -11,9 +11,13 @@ ASes:
     cert_issuer: 1-ff00:0:110
   "1-ff00:0:112": # old 1-13
     cert_issuer: 1-ff00:0:110
+  "1-ff00:0:113":
+    cert_issuer: 1-ff00:0:110
 links:
   - {a: "1-ff00:0:110#1", b: "1-ff00:0:111#41", linkAtoB: CHILD, mtu: 1280}
-  - {a: "1-ff00:0:110#2", b: "1-ff00:0:112#1", linkAtoB: CHILD, bw: 500}
+  - {a: "1-ff00:0:110#2", b: "1-ff00:0:111#42", linkAtoB: CHILD, mtu: 1280}
+  - {a: "1-ff00:0:111#2", b: "1-ff00:0:112#1", linkAtoB: CHILD, bw: 500}
+  - {a: "1-ff00:0:111#3", b: "1-ff00:0:113#1", linkAtoB: CHILD, bw: 500}
 CAs:
   CA1-1:
     ISD: 1

showpaths output:
ubuntu@ubuntu-xenial:~/go/src/github.com/scionproto/scion$ $SC/bin/showpaths -srcIA 1-ff00:0:112 -sciond /run/shm/sciond/sd1-ff00_0_112.sock -dstIA 1-ff00:0:113 Available paths to 1-ff00:0:113 [ 0] Hops: [1-ff00:0:112 1>2 1-ff00:0:111 3>1 1-ff00:0:113] Mtu: 1472 [ 1] Hops: [1-ff00:0:112 1>2 1-ff00:0:111 3>1 1-ff00:0:113] Mtu: 1472 [ 2] Hops: [1-ff00:0:112 1>2 1-ff00:0:111 3>1 1-ff00:0:113] Mtu: 1472 [ 3] Hops: [1-ff00:0:112 1>2 1-ff00:0:111 3>1 1-ff00:0:113] Mtu: 1472

expected showpaths output:
ubuntu@ubuntu-xenial:~/go/src/github.com/scionproto/scion$ $SC/bin/showpaths -srcIA 1-ff00:0:112 -sciond /run/shm/sciond/sd1-ff00_0_112.sock -dstIA 1-ff00:0:113 Available paths to 1-ff00:0:113 [ 0] Hops: [1-ff00:0:112 1>2 1-ff00:0:111 3>1 1-ff00:0:113] Mtu: 1472

@FR4NK-W
Copy link
Contributor Author

FR4NK-W commented Feb 12, 2019

Proposed fix in PR #2445
could also be moved to FilterLongPaths, filtering all useless paths

@scrye
Copy link
Contributor

scrye commented Dec 30, 2020

The definition of duplicate paths has changed slightly following the new data-plane path format, but #3930 should now eliminate duplicates. Closing this, but feel free to reopen if there are still some duplicates causing issues.

@scrye scrye closed this as completed Dec 30, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants