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

prefix-routing-consistency widget only shows a result for the route observed by the lowest amount of peers #9

Open
nusenu opened this issue Oct 8, 2018 · 8 comments

Comments

@nusenu
Copy link
Owner

nusenu commented Oct 8, 2018

example:
https://stat.ripe.net/widget/prefix-overview#w.resource=147.171.0.0%2F16

and

https://stat.ripe.net/widget/routing-history#w.resource=147.171.0.0%2F16
say the source is AS1942:

147-2

but prefix-routing-consistency disagrees. It says the only source is AS2200:

https://stat.ripe.net/widget/prefix-routing-consistency#w.resource=147.171.0.0%2F16
147

Rene Wilhelm wrote on the routing-wg (2018-09-27):
https://www.ripe.net/ripe/mail/archives/routing-wg/2018-September/003630.html

prefix-routing-consistency widget appears to indeed have problems;
at least in this case. It only shows a result for the route observed
by the lowest amount of peers.

@cteusche-ripencc
Copy link

I consider this a bug and I will check it in the next bug fixing phase.

@nusenu
Copy link
Owner Author

nusenu commented Nov 15, 2018 via email

@cteusche-ripencc
Copy link

Hi Nusenu,

I was under the impression that was fixed, as Prefix Routing Consistency also shows both ASNs for the given prefix.
Please share the resource and I'll check it!

@cteusche-ripencc
Copy link

screen shot 2018-11-16 at 11 22 49

@nusenu
Copy link
Owner Author

nusenu commented Nov 16, 2018 via email

@cteusche-ripencc
Copy link

Gotcha! I'll get back to you soon

@cteusche-ripencc
Copy link

Bummer, this issue was indeed not fixed and the second ASN in the 147.171/16 result was actually coming from the registry data, sorry about that.

I created a bug fix, which should be available in production in about 2 hours.

@nusenu
Copy link
Owner Author

nusenu commented Nov 16, 2018 via email

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