blixt icon indicating copy to clipboard operation
blixt copied to clipboard

feat: implement checksum

Open jokestax opened this issue 1 year ago • 4 comments

Description

  • Use bpf_l3_csum_replace and bpf_l4_csum_replace instead of bpf_csum_diff.
  • Implement checksum updates for both IP and TCP headers after modifying source IP/port.

jokestax avatar Dec 21 '24 08:12 jokestax

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: jokestax Once this PR has been reviewed and has the lgtm label, please assign shaneutt for approval. For more information see the Kubernetes Code Review Process.

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment Approvers can cancel approval by writing /approve cancel in a comment

k8s-ci-robot avatar Dec 21 '24 08:12 k8s-ci-robot

Heya :wave: I noticed this was draft, were you needing some more time for changes before reviews or did you want review while in draft?

shaneutt avatar Jan 06 '25 15:01 shaneutt

Hello,I’d appreciate a review on this draft to ensure I’m heading in the right direction before proceeding with similar changes for ICMP

jokestax avatar Jan 06 '25 15:01 jokestax

Hey @jokestax! I know you updated some of the other PRs, did you need further assistance with this one? (if it's just a matter of time, no sweat just wanted to be sure to help keep you unblocked)

shaneutt avatar Feb 14 '25 19:02 shaneutt

At this point we're going to consider this one closed due to inactivity, but we're happy to re-open it again later :+1:

shaneutt avatar Apr 01 '25 13:04 shaneutt