Skip to content

Add unit tests for neighbor (ARP) management #10341

@brb

Description

@brb

Currently, we manually populate neigh (ARP) entries for Cilium nodes (pkg/datapath/linux/node.go) to facilitate fib_lookup calls in the datapath.

However, we don't have any unit tests which would check the whole lifecycle of entries: adding a node -> doing arping -> inserting the entry -> removing the node -> removing the entry.

Ideally, we should not mock arping. Instead, we should create a netns, connect via a veth pair, assign IPs and then do the arping.

Metadata

Metadata

Assignees

Labels

area/CI-improvementTopic or proposal to improve the Continuous Integration workflowgood-first-issueGood starting point for new developers, which requires minimal understanding of Cilium.help-wantedPlease volunteer for this by adding yourself as an assignee!kind/featureThis introduces new functionality.pinnedThese issues are not marked stale by our issue bot.

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions