Feature/reporting 311 add msr asn peers

Ready for merge (pending additional functionality wrt caching and NRENs)

requested review from @bjarke.madsen

assigned to @sam.roberts

added 1 commit

• 1a5861b1 - unit tests added, need to fix bug causing error in tests before these are reached

Compare with previous version

added 1 commit

• 07db6a02 - all code paths in endpoint covered by unit tests

Compare with previous version

added 1 commit

• 05d8b71d - update doc index

Compare with previous version

@erik.reid How feasible do you think caching could be with this endpoint? I'm a little concerned given the variation in requests it could lead to a lot of cached requests that aren't made very often, but I can definitely see the value in caching the most common use case according to Bjarke (no filters, all ASNs in the ASN-NREN map)

resolved all threads

changed the description

We should cache everything. Just come up with some encoding for generating keys that include the parameters. We have plenty of space in memory for these all cached results, and it would make the code complicated and less clear to cache sometimes and not cache other times based on some guessed optimizations.

added 2 commits

• 7936f4a3 - rewritten to provide NREN, and to use caching
• d2994a77 - adjusted tests to work with providing NRENs

Compare with previous version

added 1 commit

• 94774acb - fixed a bug involving ASN not being presented as integer correctly

Compare with previous version

added 1 commit

• a5822127 - remove unused import to make tox happy

Compare with previous version

Caching and NREN return added, all tests adjusted, coverage for endpoint is complete and Bjarke thinks this all looks fine, so I'm going to go ahead and merge this.

mentioned in commit 3f8eb4ea

merged