Skip to content

Refactor code of main.go#340

Merged
brb merged 3 commits intocilium:mainfrom
Asphaltt:refactor/main.go
Jun 17, 2024
Merged

Refactor code of main.go#340
brb merged 3 commits intocilium:mainfrom
Asphaltt:refactor/main.go

Conversation

@Asphaltt
Copy link
Contributor

@Asphaltt Asphaltt commented Mar 4, 2024

Refactor code of attaching kprobe and attaching skb-tracking kprobe.

Signed-off-by: Leon Hwang hffilwlqm@gmail.com

@Asphaltt Asphaltt requested a review from a team as a code owner March 4, 2024 15:07
@Asphaltt Asphaltt requested review from brb and removed request for a team March 4, 2024 15:07
@Asphaltt Asphaltt mentioned this pull request Apr 6, 2024
Hide attaching kprobes progress from `main.go`.

Signed-off-by: Leon Hwang <hffilwlqm@gmail.com>
Hide the attaching details of tracking skb from `main.go`.

Signed-off-by: Leon Hwang <hffilwlqm@gmail.com>
Hide attaching kprobe to non-skb funcs from `main.go`.

Signed-off-by: Leon Hwang <hffilwlqm@gmail.com>
@brb
Copy link
Member

brb commented Jun 11, 2024

Hey @Asphaltt , is it ready for the review?

@Asphaltt
Copy link
Contributor Author

Yes, it is ready.

Copy link
Member

@brb brb left a comment

Choose a reason for hiding this comment

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

Thanks! Two nits (can be addressed in follow-up PRs).

@brb brb merged commit 3871a44 into cilium:main Jun 17, 2024
Asphaltt added a commit to Asphaltt/pwru that referenced this pull request Jun 17, 2024
Address comments in PR cilium#340.

Signed-off-by: Leon Hwang <hffilwlqm@gmail.com>
Asphaltt added a commit to Asphaltt/pwru that referenced this pull request Jul 7, 2024
Address comments in PR cilium#340.

Signed-off-by: Leon Hwang <hffilwlqm@gmail.com>
brb pushed a commit that referenced this pull request Jul 7, 2024
Address comments in PR #340.

Signed-off-by: Leon Hwang <hffilwlqm@gmail.com>
P3gEcTH8uJiYFVKb7lIn added a commit to P3gEcTH8uJiYFVKb7lIn/pwru that referenced this pull request Oct 5, 2025
Address comments in PR cilium/pwru#340.

Signed-off-by: Leon Hwang <hffilwlqm@gmail.com>
dawn9444-holloway5 added a commit to dawn9444-holloway5/pwru that referenced this pull request Oct 10, 2025
Address comments in PR cilium/pwru#340.

Signed-off-by: Leon Hwang <hffilwlqm@gmail.com>
otpfmabtmksfrk6931CesarWolfe added a commit to otpfmabtmksfrk6931CesarWolfe/pwru that referenced this pull request Oct 12, 2025
Address comments in PR cilium/pwru#340.

Signed-off-by: Leon Hwang <hffilwlqm@gmail.com>
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.

2 participants