Skip to content

Conversation

tklauser
Copy link
Member

@tklauser tklauser commented Mar 21, 2025

Add feature probes for map types BloomFilter, UserRingbuf, CgroupStorage and Arena and for prog type Netfilter.

@tklauser tklauser requested a review from rgo3 as a code owner March 21, 2025 13:34
@tklauser tklauser requested review from lmb and removed request for rgo3 March 21, 2025 13:35
@tklauser tklauser changed the title features: update map feature matrix for types added since 5.11 features: update map/prog feature matrix for types added since 5.11 Mar 21, 2025
@tklauser tklauser changed the title features: update map/prog feature matrix for types added since 5.11 features: update map/prog feature matrix for newly added types Mar 21, 2025
@tklauser tklauser changed the title features: update map/prog feature matrix for newly added types Add new ≥ 5.11 map/prog types to feature matrix Mar 21, 2025
@lmb lmb requested a review from rgo3 March 24, 2025 09:39
…amHelper

Make sure the cases fore these program types in haveProgramHelper are
covered by TestHaveProgramHelper.

Note that the only new helper FnSendSignal added in 5.3 is not usable in
CGroupSockopt programs, thus fall back to FnSkStorageDelete which was
added in 5.2.

Suggested-by: Timo Beckers <timo@isovalent.com>
Signed-off-by: Tobias Klauser <tobias@cilium.io>
Add feature probes for map types BloomFilter, UserRingbuf, CgroupStorage
and Arena.

Signed-off-by: Tobias Klauser <tobias@cilium.io>
Add feature probe for prog type Netfilter.

Signed-off-by: Tobias Klauser <tobias@cilium.io>
@tklauser tklauser force-pushed the pr/tklauser/features-map-update branch from 917eab9 to 231a9fd Compare March 25, 2025 10:30
@tklauser tklauser requested review from ti-mo and lmb March 25, 2025 10:32
@ti-mo
Copy link
Collaborator

ti-mo commented Mar 25, 2025

Excellent, many thanks!

@ti-mo ti-mo merged commit 188fb43 into cilium:main Mar 25, 2025
17 checks passed
@tklauser tklauser deleted the pr/tklauser/features-map-update branch March 25, 2025 12:27
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.

4 participants