Merged
Conversation
leodido
changed the title
block_ip to block_ipv4
leodido
force-pushed
the
rename/block-ip-to-block-ipv4
branch
from
6442cff to
a8836be
Compare
Rename the BPF source file, function name, and bpf_printk strings. The program only handles IPv4 — the new name makes that explicit. Co-authored-by: Ona <no-reply@ona.com>
Update input_fields key in api.lua and enum references in input_parse.h to match the renamed BPF program. Co-authored-by: Ona <no-reply@ona.com>
Update CLI examples, CNI examples, and built-in programs descriptions in README.txt and docs/README.md. Co-authored-by: Ona <no-reply@ona.com>
Update test names, commands, assertions, and CNI fixture to use the new program name. Co-authored-by: Ona <no-reply@ona.com>
leodido
force-pushed
the
rename/block-ip-to-block-ipv4
branch
from
a8836be to
9886730
Compare
leodido
added a commit
that referenced
this pull request
May 3, 2026
The program only handles IPv4 (ETH_P_IP). The new name makes the scope unambiguous, consistent with the block_ipv4 rename in PR #46. Co-authored-by: Ona <no-reply@ona.com>
leodido
added a commit
that referenced
this pull request
May 3, 2026
The program only handles IPv4 (ETH_P_IP). The new name makes the scope unambiguous, consistent with the block_ipv4 rename in PR #46. Co-authored-by: Ona <no-reply@ona.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
block_iponly inspects IPv4 (ETH_P_IP) and filters by IPv4 destination address. The name implies generic IP handling but it doesn't touch IPv6. Renaming toblock_ipv4makes the scope unambiguous.This establishes the naming convention for the
allow_ipv4rename in PR #41.Changes
bpf/block_ip.bpf.c→bpf/block_ipv4.bpf.c(file, function,bpf_printkstrings)api.luainput_fields keyinput_parse.henum referencesREADME.txtanddocs/README.mdPure rename — no behavioral changes.