cilium icon indicating copy to clipboard operation
cilium copied to clipboard

experimental: Benchmark reconciling tables and BPF

Open DamianSawicki opened this issue 1 year ago • 3 comments
trafficstars

Add a benchmark for addition of k8s services and endpoints, reflecting them to StateDB tables, and reconciliation to BPF maps.


Please ensure your pull request adheres to the following guidelines:

  • [x] For first time contributors, read Submitting a pull request
  • [x] All code is covered by unit and/or runtime tests where feasible.
  • [x] All commits contain a well written commit description including a title, description and a Fixes: #XXX line if the commit addresses a particular GitHub issue.
  • [x] If your commit description contains a Fixes: <commit-id> tag, then please add the commit author[s] as reviewer[s] to this issue.
  • [x] All commits are signed off. See the section Developer’s Certificate of Origin
  • [x] Provide a title or release-note blurb suitable for the release notes.
  • [x] Are you a user of Cilium? Please add yourself to the Users doc
  • [x] Thanks for contributing!

DamianSawicki avatar Aug 21 '24 11:08 DamianSawicki

/assign joamaki

DamianSawicki avatar Aug 21 '24 11:08 DamianSawicki

cc @bowei

robscott avatar Aug 21 '24 15:08 robscott

some minor comments.

You may want to consider having two commits -- one move and another that does diffs.

Thanks for review @bowei! I've split this into two commits as you suggested.

DamianSawicki avatar Aug 26 '24 11:08 DamianSawicki

/test

joamaki avatar Sep 16 '24 08:09 joamaki

Commit 1135b9bac1e689fdb4c09e1933f397d5245959bb does not match "(?m)^Signed-off-by:".

Please follow instructions provided in https://docs.cilium.io/en/stable/contributing/development/contributing_guide/#developer-s-certificate-of-origin

/test

joamaki avatar Sep 17 '24 09:09 joamaki

/test

joamaki avatar Sep 19 '24 15:09 joamaki