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

Performance job for NFTables #31620

Closed
aroradaman opened this issue Jan 13, 2024 · 6 comments · Fixed by #32431
Closed

Performance job for NFTables #31620

aroradaman opened this issue Jan 13, 2024 · 6 comments · Fixed by #32431
Labels
sig/network Categorizes an issue or PR as relevant to SIG Network. sig/scalability Categorizes an issue or PR as relevant to SIG Scalability.

Comments

@aroradaman
Copy link
Member

aroradaman commented Jan 13, 2024

There were discussions in NFTables KEP regarding performance comparison with IPTables proxy.

We are looking out for a perf job for NFTables proxy but aren't sure about the size/scale of the cluster, load on the cluster, frequency of the job, and other things. Along with comparing the perf with IPTables, this will also help us compare perf with possible future optimizations within NFTables proxy.

ci-kubernetes-e2e-gce-scale-performance tracks NetworkLatency and NetworkProgrammingLatency for IPTables proxy at scale, maybe we need similar metrics at a low scale?

Opening up this issue for discussion.

/sig network
/sig scalability

/cc. @danwinship @aojea @thockin

@k8s-ci-robot k8s-ci-robot added the needs-sig Indicates an issue or PR lacks a `sig/foo` label and requires one. label Jan 13, 2024
@aroradaman
Copy link
Member Author

/cc. @ameukam

@aroradaman
Copy link
Member Author

/sig network
/sig scalability

@k8s-ci-robot k8s-ci-robot added sig/network Categorizes an issue or PR as relevant to SIG Network. sig/scalability Categorizes an issue or PR as relevant to SIG Scalability. and removed needs-sig Indicates an issue or PR lacks a `sig/foo` label and requires one. labels Jan 13, 2024
@aojea
Copy link
Member

aojea commented Jan 13, 2024

@npinaeva and I are working on this, no need to go to scale in a first iteration, we want to validate first the improvements of nftables over the well known iptables problems:, that are:

  • programming latency: iptables-save and restore with large numbers of rules
  • latency degradation vs number of rules caused by the linear search on the iptables rules

@k8s-triage-robot
Copy link

The Kubernetes project currently lacks enough contributors to adequately respond to all issues.

This bot triages un-triaged issues according to the following rules:

  • After 90d of inactivity, lifecycle/stale is applied
  • After 30d of inactivity since lifecycle/stale was applied, lifecycle/rotten is applied
  • After 30d of inactivity since lifecycle/rotten was applied, the issue is closed

You can:

  • Mark this issue as fresh with /remove-lifecycle stale
  • Close this issue with /close
  • Offer to help out with Issue Triage

Please send feedback to sig-contributor-experience at kubernetes/community.

/lifecycle stale

@k8s-ci-robot k8s-ci-robot added the lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. label Apr 12, 2024
@npinaeva
Copy link
Member

/remove-lifecycle stale

@k8s-ci-robot k8s-ci-robot removed the lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. label Apr 15, 2024
@aojea
Copy link
Member

aojea commented Apr 15, 2024

#32431

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
sig/network Categorizes an issue or PR as relevant to SIG Network. sig/scalability Categorizes an issue or PR as relevant to SIG Scalability.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants