Skip to content

Conversation

@ties
Copy link
Contributor

@ties ties commented Nov 11, 2025

Use the originated time instead of the MRT header time when processing MRT TableDump (v2) messages into BgpElem in Elementor.

Fixes #251

Use the originated time instead of the MRT header time when
processing MRT TableDump (v2) messages into BgpElem in Elementor.
@codecov
Copy link

codecov bot commented Nov 11, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 90.91%. Comparing base (0453969) to head (3927066).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #252      +/-   ##
==========================================
+ Coverage   90.87%   90.91%   +0.03%     
==========================================
  Files          80       80              
  Lines       12588    12588              
==========================================
+ Hits        11439    11444       +5     
+ Misses       1149     1144       -5     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@ties
Copy link
Contributor Author

ties commented Nov 11, 2025

Looking at a parquet file processed using this branch of bgpkit-parser, the file contains the dates of the restarts of the various collectors as lower bound for the timestamp, and has an upper bound right before the snapshot was made.

D select min(ts), max(ts), collector from '20251011.0000-bview-all-collectors.parquet' group by all order by 3;
┌─────────────────────┬─────────────────────┬───────────┐
│       min(ts)       │       max(ts)       │ collector │
│      timestamp      │      timestamp      │  varchar  │
├─────────────────────┼─────────────────────┼───────────┤
│ 2025-06-18 10:50:44 │ 2025-10-10 23:59:59 │ rrc00     │
│ 2025-06-26 09:25:05 │ 2025-10-10 23:59:59 │ rrc01     │
│ 2025-06-19 09:24:45 │ 2025-10-10 23:59:59 │ rrc03     │
│ 2025-07-22 11:43:19 │ 2025-10-10 23:59:59 │ rrc04     │
│ 2025-06-05 09:40:50 │ 2025-10-10 23:59:59 │ rrc05     │
│ 2025-08-27 07:58:01 │ 2025-10-10 23:59:59 │ rrc06     │
│ 2025-08-08 12:13:42 │ 2025-10-10 23:59:59 │ rrc07     │
│ 2025-06-12 09:35:10 │ 2025-10-10 23:59:59 │ rrc10     │
│ 2025-06-11 10:26:18 │ 2025-10-10 23:59:59 │ rrc11     │
│ 2025-06-26 12:10:51 │ 2025-10-10 23:59:59 │ rrc12     │
│ 2025-06-26 09:53:30 │ 2025-10-10 23:59:59 │ rrc13     │
│ 2025-07-22 12:21:26 │ 2025-10-10 23:59:59 │ rrc14     │
│ 2025-09-01 14:36:33 │ 2025-10-10 23:59:59 │ rrc15     │
│ 2025-06-25 10:13:47 │ 2025-10-10 23:59:59 │ rrc16     │
│ 2025-06-10 10:06:58 │ 2025-10-10 23:59:51 │ rrc18     │
│ 2025-08-07 09:02:44 │ 2025-10-10 23:59:59 │ rrc19     │
│ 2025-08-08 11:54:44 │ 2025-10-10 23:59:59 │ rrc20     │
│ 2025-08-07 09:34:30 │ 2025-10-10 23:59:59 │ rrc21     │
│ 2025-08-28 21:11:12 │ 2025-10-10 23:59:59 │ rrc22     │
│ 2025-07-22 12:29:43 │ 2025-10-10 23:59:59 │ rrc23     │
│ 2025-09-07 16:49:00 │ 2025-10-10 23:59:59 │ rrc24     │
│ 2025-08-07 11:24:13 │ 2025-10-10 23:59:59 │ rrc25     │
│ 2025-07-22 11:49:17 │ 2025-10-10 23:59:59 │ rrc26     │
├─────────────────────┴─────────────────────┴───────────┤
│ 23 rows                                     3 columns │
└───────────────────────────────────────────────────────┘

@digizeph digizeph merged commit 4717050 into bgpkit:main Nov 11, 2025
4 checks passed
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

Successfully merging this pull request may close these issues.

Use originated time from MRT peer table RIB entries

2 participants