WARNING: THIS SITE IS A MIRROR OF GITHUB.COM / IT CANNOT LOGIN OR REGISTER ACCOUNTS / THE CONTENTS ARE PROVIDED AS-IS / THIS SITE ASSUMES NO RESPONSIBILITY FOR ANY DISPLAYED CONTENT OR LINKS / IF YOU FOUND SOMETHING MAY NOT GOOD FOR EVERYONE, CONTACT ADMIN AT ilovescratch@foxmail.com
Skip to content

[Chores]: Conntracker code refactoring and chores #156

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.

Sign up for GitHub

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

Jump to bottom
Merged
LorenzoTettamanti merged 3 commits into from
Nov 12, 2025
Merged

[Chores]: Conntracker code refactoring and chores #156

LorenzoTettamanti merged 3 commits into from
Nov 12, 2025

Conversation

@LorenzoTettamanti
Copy link
Member

@LorenzoTettamanti LorenzoTettamanti commented Nov 11, 2025 •
edited
Loading

Changes:

This PR introduces very few changes before opening a PR for the #150 issue. No code functionalities were changed.

Changes:

  • Removed bindings.rs from conntracker and metrics-tracer.rs : users/developers can generate bindings with bindgen using the build-conntracker.sh script
  • Added documentation to all the conntracker functions
  • Improved conntracker general modules organization

Tests:

No tests required

@LorenzoTettamanti LorenzoTettamanti self-assigned this Nov 11, 2025
@LorenzoTettamanti LorenzoTettamanti added the documentation Improvements or additions to documentation label Nov 11, 2025
@LorenzoTettamanti LorenzoTettamanti added code refactoring Code refactoring rust Pull requests that update Rust code labels Nov 11, 2025
@LorenzoTettamanti LorenzoTettamanti moved this to In Progress in CortexFlow v0.1 Nov 11, 2025
@LorenzoTettamanti LorenzoTettamanti changed the title Conntracker code refactoring and chores [Chores]: Conntracker code refactoring and chores Nov 11, 2025
@LorenzoTettamanti LorenzoTettamanti marked this pull request as ready for review November 11, 2025 17:56
@siddh34
Copy link
Collaborator

siddh34 commented Nov 12, 2025

image

Your changes are working mate @LorenzoTettamanti 🚀🚀

@LorenzoTettamanti LorenzoTettamanti merged commit 4c68ba2 into CortexFlow:feature/ebpf-core Nov 12, 2025
3 checks passed
@LorenzoTettamanti LorenzoTettamanti moved this from In Progress to Done in CortexFlow v0.1 Nov 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@lorebrada lorebrada Awaiting requested review from lorebrada lorebrada is a code owner

1 more reviewer

@siddh34 siddh34 siddh34 approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@LorenzoTettamanti LorenzoTettamanti

Labels

code refactoring Code refactoring documentation Improvements or additions to documentation rust Pull requests that update Rust code

Projects

CortexFlow v0.1
Status: Done

Milestone

CortexBrain core v 0.1.0

Development

Successfully merging this pull request may close these issues.

2 participants

@LorenzoTettamanti @siddh34