8000 Upgrade Keycloak's sizing guide for KC26 by ahus1 · Pull Request #32344 · keycloak/keycloak · GitHub
[go: up one dir, main page]

Skip to content

Upgrade Keycloak's sizing guide for KC26 - #32344

Merged
mhajas merged 3 commits into
keycloak:mainfrom
ahus1:is-32343-update-kecloak-sizing-guide
Aug 27, 2024
Merged

mhajas merged 3 commits into
keycloak:mainfrom
ahus1:is-32343-update-kecloak-sizing-guide

Conversation

@ahus1
@ahus1 ahus1 commented Aug 22, 2024
Copy link
Copy Markdown
Member

Closes #32343

@ahus1 ahus1 self-assigned this Aug 22, 2024
Closes keycloak#32343

Signed-off-by: Alexander Schwartz <aschwart@redhat.com>
@ahus1
ahus1 force-pushed the is-32343-update-kecloak-sizing-guide branch from 22f2b96 to 6298ad4 Compare August 22, 2024 13:11
@ahus1
ahus1 requested a review from kami619 August 26, 2024 13:26
@ahus1
ahus1 commented Aug 26, 2024
Copy link
Copy Markdown
Member Author

@kami619 - please review when you have the time. Thanks!

@ahus1
ahus1 marked this pull request as ready for review August 26, 2024 13:26
@ahus1
ahus1 requested a review from a team as a code owner August 26, 2024 13:26
To calculate the requested memory, use the calculation above. As memory limit, subtract the non-heap memory from the value above and divide the result by 0.7.

* For each 45 password-based user logins per second, 1 vCPU per Pod in a three-node cluster (tested with up to 300 per second).
* For each 15 password-based user logins per second, allocate 1 vCPU to the cluster (tested with up to 300 per second).
8000
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Would this then compute to, 135 user logins per second, if only the user login scenario is load tested against the 3 pod cluster in an Active/Active setup ? @ahus1

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In a 2x3 Pods A/A setup and 135 logins per seconds, you would see 1.5 vCPUs used on each pod when both clusters are active. If then one site goes down, you would then see the usage of 3.0 vCPUs on each of the three remaining active Pods.

I'd think you have a choice here then either request 1.5 vCPUs per Pod and rely on the cluster to have enough bandwidth when one site goes down, or request 3.0 vCPUs per Pod to be on the safe side. Maybe request 1.5 vCPUs and then have an auto-scaler scale up 6 Pod in the remaining DC if you can afford the delay of an autoscaler on the pod level and the cluster level. IMHO there is no simple answer to that. I'll see if I can provide an answer on this, still.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you Alex, this helps!

ahus1 and others added 2 commits August 27, 2024 12:27
Co-authored-by: Kamesh Akella <kakella@redhat.com>
Signed-off-by: Alexander Schwartz <alexander.schwartz@gmx.net>
Closes keycloak#32343

Signed-off-by: Alexander Schwartz <aschwart@redhat.com>
@ahus1
ahus1 requested a review from kami619 August 27, 2024 12:09
@ahus1
ahus1 commented Aug 27, 2024
Copy link
Copy Markdown
Member Author

@kami619 - I've pushed a change. The part that talks about specific DB instances is now moved to the example section, while the performance recommendations are then more general. I also described three alternatives on how to do a sizing of a multi-site setup with regards to the CPUs.

I'm looking forward to your review.

@kami619
kami619 commented Aug 27, 2024
Copy link
Copy Markdown
Contributor

@ahus1 Thanks for the changes. It looks good to me now.

@ahus1
ahus1 requested a review from mhajas August 27, 2024 14:29
@ahus1
ahus1 commented Aug 27, 2024
Copy link
Copy Markdown
Member Author

@mhajas - given @kami619's review, can you please approve and merge it? Thanks!

@ahus1
ahus1 removed the request for review from kami619 August 27, 2024 14:29
@mhajas mhajas left a comment
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approving based on the review from @kami619

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.

Upgrade Keycloak's sizing guide for KC26 and persistent sessions

3 participants

0