Skip to content

Monitor Aggregation for connections does not work with host firewall #12561

@joestringer

Description

@joestringer

The host firewall does not respect the monitor parameter to the ct_lookup[46]() functions, it simply gathers it and discards, for example:

__u32 monitor = 0, dst_id = WORLD_ID;

ret = ct_lookup4(get_ct_map4(&tuple), &tuple, ctx, l4_off, CT_INGRESS,
&ct_state, &monitor);

This means that even with monitorAggregationLevel=medium, aggregation of monitor notifications based on connections does not occur correctly for packets from the host towards the networks, meaning that notifications are generated once per packet rather than once every 5s per connection.

The monitor argument should be handled similarly to how it is used in bpf/bpf_lxc.c in the bpf_host.c.

Metadata

Metadata

Assignees

Labels

area/datapathImpacts bpf/ or low-level forwarding details, including map management and monitor messages.area/host-firewallImpacts the host firewall or the host endpoint.area/monitorImpacts monitoring, access logging, flow logging, visibility of datapath traffic.good-first-issueGood starting point for new developers, which requires minimal understanding of Cilium.

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions