-
Notifications
You must be signed in to change notification settings - Fork 1.1k
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
Selenium Grid Scaler should consider "maxSession" #2618
Comments
Seems reasonable, are you willing to contribute this? |
Sorry, but unfortunately I'm merely a user and not a developer. |
I can take this one. |
Thanks a lot! ❤️ |
@adborroto Any update on this issue |
PR open |
@JorTurFer / @adborroto I see this issue is closed... Is there any version we need to get this |
Hi @amit337496 |
@JorTurFer Thank you.. I am able to see changes in 2.7 version. How this scale down will happen in this scenario.. If node is still running will it kill that? |
hi @amit337496 |
My scenario is I have 10 test cases and I set max nodes to 5. So it scaled up 5 servers and execution started... But for period of time scale down happened even if they are active sessions |
Did you check ScaledJob? It, s the solution for long term processes |
Proposal
The graphql API of Selenium Grid allows to query the "maxSession" value per grid and/or node:
https://www.selenium.dev/documentation/grid/advanced_features/graphql_support/#querying-the-number-of-maxsession-and-sessioncount-in-the-grid-
That is, if maxSession=5 each node can hold up to 5 concurrent sessions. Therefore, if there are 10 requests in the queue, only 2 nodes are neccessary to fulfill the requirements. Currently, the Selenium Grid Scaler of KEDA will scale up to 10 nodes in this case, ignoring the maxSessions per node.
Use-Case
No response
Anything else?
No response
The text was updated successfully, but these errors were encountered: