Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: cilium network load test pipeline #540

Draft
wants to merge 67 commits into
base: main
Choose a base branch
from

Conversation

huntergregory
Copy link

@huntergregory huntergregory commented Feb 21, 2025

Goal

Validate scenario with induce heavy pod traffic on many nodes.

Validations

TODO: implement

  1. cilium pod is ok.
  2. Datapath is ok (e.g. conntrack).
  3. Network observability (Hubble) metrics are ok.

Validations

TODO: implement

Topology

Topology when nodes=2, serversPerNode=2, clientsPerNode=2, namespaces=4, deployments=2.

You can specify queries per second coming from each client as well.

image

Topology Parameters

  • nodes
  • servers per node
  • clients per node
  • queries per second (per client)
  • namespaces. There is 1 service per namespace, and all server deployments in the namespace are its backend pods.
  • deployments. A value of N means N client deployments in the cluster, each talking to a corresponding server deployment. Controls the number of cilium identities.

Dependencies

Requires pending CL2 support for scraping hubble metrics.

Implementation Notes

Other Parameters

  • whether to apply a CNP that generates DNS metrics (requires ACNS security agent)

Signed-off-by: Hunter Gregory <[email protected]>
Signed-off-by: Hunter Gregory <[email protected]>
Signed-off-by: Hunter Gregory <[email protected]>
Signed-off-by: Hunter Gregory <[email protected]>
Signed-off-by: Hunter Gregory <[email protected]>
@huntergregory huntergregory force-pushed the huntergregory/scenario-cilium-network-load branch from facb24c to 6afcedb Compare February 26, 2025 19:35
@huntergregory huntergregory force-pushed the huntergregory/scenario-cilium-network-load branch from be380ee to 863a9df Compare February 26, 2025 20:27
Signed-off-by: Hunter Gregory <[email protected]>
Signed-off-by: Hunter Gregory <[email protected]>
Signed-off-by: Hunter Gregory <[email protected]>
Signed-off-by: Hunter Gregory <[email protected]>
Signed-off-by: Hunter Gregory <[email protected]>
Signed-off-by: Hunter Gregory <[email protected]>
Signed-off-by: Hunter Gregory <[email protected]>
Signed-off-by: Hunter Gregory <[email protected]>
Signed-off-by: Hunter Gregory <[email protected]>
… now and comment out deletes

Signed-off-by: Hunter Gregory <[email protected]>
Signed-off-by: Hunter Gregory <[email protected]>
Signed-off-by: Hunter Gregory <[email protected]>
Signed-off-by: Hunter Gregory <[email protected]>
Signed-off-by: Hunter Gregory <[email protected]>
Signed-off-by: Hunter Gregory <[email protected]>
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.

1 participant