inex / IXP-Manager

Full stack web application powering peering at over 200 Internet Exchange Points (IXPs) globally.
https://www.ixpmanager.org/
GNU General Public License v2.0
375 stars 160 forks source link

Regression: MRTG Graphs for reseller uplink ports are no longer generated. #860

Closed ichilton closed 1 year ago

ichilton commented 1 year ago

Commit https://github.com/inex/IXP-Manager/commit/e40be75de9deec22978df766dee88b912242316b, which was to fix https://github.com/inex/IXP-Manager/issues/855 contains a regression: it no longer generates graphs for reseller uplink ports.

It should generate port graphs for reseller uplink ports, it just shouldn't sum reseller uplink ports in the aggregates.

barryo commented 1 year ago

@ichilton - pull from release-v6 to get 4420a5905e0b310c7703a6d3002422c3654fc400 which should fix it I think. Seems to work for INEX.

ichilton commented 1 year ago

@barryo Thank you very much! - will pull now and test!