-
Notifications
You must be signed in to change notification settings - Fork 3.4k
Closed
Labels
area/CI-improvementTopic or proposal to improve the Continuous Integration workflowTopic or proposal to improve the Continuous Integration workflowgood-first-issueGood starting point for new developers, which requires minimal understanding of Cilium.Good starting point for new developers, which requires minimal understanding of Cilium.help-wantedPlease volunteer for this by adding yourself as an assignee!Please volunteer for this by adding yourself as an assignee!kind/featureThis introduces new functionality.This introduces new functionality.pinnedThese issues are not marked stale by our issue bot.These issues are not marked stale by our issue bot.
Description
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 workflowTopic or proposal to improve the Continuous Integration workflowgood-first-issueGood starting point for new developers, which requires minimal understanding of Cilium.Good starting point for new developers, which requires minimal understanding of Cilium.help-wantedPlease volunteer for this by adding yourself as an assignee!Please volunteer for this by adding yourself as an assignee!kind/featureThis introduces new functionality.This introduces new functionality.pinnedThese issues are not marked stale by our issue bot.These issues are not marked stale by our issue bot.