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

Test Flux v2.3 release candidate with CRD load #6

Merged
merged 7 commits into from
May 8, 2024
Merged

Conversation

stefanprodan
Copy link
Member

@stefanprodan stefanprodan commented Apr 19, 2024

Changes to benchmark:

  • Update Flux to v2.3 (pre-releaase images and CRDs)
  • Update Kubernetes to v1.29
  • Pre-install CRDs 50/250/500 to check if Flux performance is affected by a large number of API extensions

Preliminary results:

  • Compared to Flux 2.2 and Kubernetes 1.28 we observe no performance regression in Flux 2.3 and Kubernetes 1.29
  • Compared to Flux 2.2 and Kubernetes 1.28 where large number of CRDs would drive helm-controller into OOM, in Flux 2.3 and Kubernetes 1.29, even with 500 CRDs, helm-controller reconciles 1K HelmReleases in under 9 minutes when configured with concurrent 10, 2CPU and 1GB RAM limits.

Shaked and others added 6 commits April 20, 2024 10:15
Co-authored-by: Shaked Klein Orbach <tklein.shaked@gmail.com>
Signed-off-by: Stefan Prodan <stefan.prodan@gmail.com>
Signed-off-by: Stefan Prodan <stefan.prodan@gmail.com>
Signed-off-by: Stefan Prodan <stefan.prodan@gmail.com>
Signed-off-by: Stefan Prodan <stefan.prodan@gmail.com>
Signed-off-by: Stefan Prodan <stefan.prodan@gmail.com>
Signed-off-by: Stefan Prodan <stefan.prodan@gmail.com>
Signed-off-by: Stefan Prodan <stefan.prodan@gmail.com>
@stefanprodan stefanprodan changed the title Test Flux v2.3 release candidate Test Flux v2.3 release candidate with CRD load May 8, 2024
@stefanprodan stefanprodan merged commit 20f3c5e into main May 8, 2024
10 checks passed
@stefanprodan stefanprodan deleted the crd-test branch May 8, 2024 08:08
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.

2 participants