-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
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: