Skip to content

FEAT: to_ingress_queue_size metric - #365

Merged
PaulRalnikov merged 19 commits into
mainfrom
fix-link-queues
Jul 30, 2025
Merged

FEAT: to_ingress_queue_size metric#365
PaulRalnikov merged 19 commits into
mainfrom
fix-link-queues

Conversation

@PaulRalnikov

@PaulRalnikov PaulRalnikov commented Jul 17, 2025 •
edited
Loading

Copy link
Copy Markdown
Collaborator

closes #366

@PaulRalnikov
PaulRalnikov marked this pull request as ready for review July 17, 2025 14:09
AntoxaBarin
AntoxaBarin reviewed Jul 18, 2025
View reviewed changes
Comment thread source/link/packet_queue/link_queue.hpp Outdated
AntoxaBarin
AntoxaBarin reviewed Jul 18, 2025
View reviewed changes
Comment thread source/metrics/write_to_csv.cpp Outdated
AntoxaBarin
AntoxaBarin reviewed Jul 18, 2025
View reviewed changes
Comment thread source/metrics/plot_metadata.hpp Outdated
PaulRalnikov and others added 2 commits July 18, 2025 14:11
@PaulRalnikov @AntoxaBarin
Update source/metrics/write_to_csv.cpp
40197b4 
Co-authored-by: Ivan Shanygin <88805084+AntoxaBarin@users.noreply.github.com>
@PaulRalnikov @AntoxaBarin
Update source/metrics/plot_metadata.hpp
710ecc6 
Co-authored-by: Ivan Shanygin <88805084+AntoxaBarin@users.noreply.github.com>
@PaulRalnikov
PaulRalnikov requested a review from AntoxaBarin July 22, 2025 11:25
ArtyomPeshkov
ArtyomPeshkov reviewed Jul 29, 2025
View reviewed changes
Comment thread source/link/packet_queue/link_queue.hpp Outdated
Comment thread source/link/link.cpp Outdated
Comment thread source/utils/filesystem.hpp
Comment thread source/metrics/write_to_csv.cpp Outdated
Comment thread source/metrics/multi_id_metrics_storage.hpp Outdated
PaulRalnikov and others added 3 commits July 29, 2025 19:22
@PaulRalnikov @ArtyomPeshkov
Apply suggestion from @ArtyomPeshkov
0365ad2 
Co-authored-by: Артём Пешков <47276660+ArtyomPeshkov@users.noreply.github.com>
@PaulRalnikov @ArtyomPeshkov
Apply suggestion from @ArtyomPeshkov
fd85b27 
Co-authored-by: Артём Пешков <47276660+ArtyomPeshkov@users.noreply.github.com>
@PaulRalnikov @ArtyomPeshkov
Apply suggestion from @ArtyomPeshkov
48c96eb 
Co-authored-by: Артём Пешков <47276660+ArtyomPeshkov@users.noreply.github.com>
ArtyomPeshkov
ArtyomPeshkov reviewed Jul 29, 2025
View reviewed changes
Comment thread source/metrics/write_to_csv.cpp Outdated
PaulRalnikov and others added 3 commits July 30, 2025 15:02
@PaulRalnikov @ArtyomPeshkov
Update source/metrics/write_to_csv.cpp
d1fb2dd 
Co-authored-by: Артём Пешков <47276660+ArtyomPeshkov@users.noreply.github.com>
@PaulRalnikov
Merge branch 'main' into fix-link-queues
c8368a4
@PaulRalnikov
Change LinkQueue constructor agrument list
6d2b13f
ArtyomPeshkov
ArtyomPeshkov approved these changes Jul 30, 2025
View reviewed changes

@ArtyomPeshkov ArtyomPeshkov left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Thanks!

@PaulRalnikov
PaulRalnikov merged commit 53f1050 into main Jul 30, 2025
6 checks passed
@PaulRalnikov
PaulRalnikov deleted the fix-link-queues branch July 30, 2025 14:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ArtyomPeshkov ArtyomPeshkov ArtyomPeshkov approved these changes
@drgulevich drgulevich Awaiting requested review from drgulevich
@Az3git Az3git Awaiting requested review from Az3git
@AntoxaBarin AntoxaBarin Awaiting requested review from AntoxaBarin

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

FEAT: add measurements of link destination device ingess buffer

3 participants

@PaulRalnikov @ArtyomPeshkov @AntoxaBarin