Skip to content
This repository has been archived by the owner on Jul 28, 2023. It is now read-only.

Use higher default pollingTimeout than 1 second #30

Open
jzaefferer opened this issue Jul 24, 2015 · 7 comments
Open

Use higher default pollingTimeout than 1 second #30

jzaefferer opened this issue Jul 24, 2015 · 7 comments

Comments

@jzaefferer
Copy link

BrowserStack implemented rate limiting for their API, currently 120 requests/min. The 1s pollingTimeout, combined with other requests, makes it likely to quickly hit that limit.

browserstack-runner has a default polling interval of 2 seconds.

@mgol
Copy link
Contributor

mgol commented Jul 27, 2015

A better solution would be to use a global pool queue instead of separate ones for each browser - current solution will always hit the limit if the number of browsers is high enough.

@forbesjo
Copy link

forbesjo commented Sep 1, 2015

+1 to finding a way around the limit

@gdubicki
Copy link

gdubicki commented Sep 6, 2015

+1

@coryvirok
Copy link

+1 - I've been talking with BrowserStack about this but I haven't gotten very far.

I get this when I run 70 tests in 2 browsers even with pollingTimeout set to 10 seconds, (as recommended by BrowserStack.)

Their rate limit seems a bit severe.

::end rant::

Looks like you can get all workers in a single API call, (GET /workers) to implement the pool option that @mzgol mentioned.

shirish87 added a commit to shirish87/karma-browserstack-launcher that referenced this issue Oct 4, 2015
Uses a shared poller to fetch and update the status of workers; helps reduce the no. of requests made to BrowserStack.
Ref karma-runner#30
shirish87 added a commit to shirish87/karma-browserstack-launcher that referenced this issue Oct 30, 2015
Uses a shared poller to fetch and update the status of workers; helps reduce the no. of requests made to BrowserStack.
Ref karma-runner#30
shirish87 added a commit to shirish87/karma-browserstack-launcher that referenced this issue Oct 30, 2015
Uses a shared poller to fetch and update the status of workers; helps reduce the no. of requests made to BrowserStack.
Ref karma-runner#30
shirish87 added a commit to shirish87/karma-browserstack-launcher that referenced this issue Oct 31, 2015
Uses a shared poller to fetch and update the status of workers; helps reduce the no. of requests made to BrowserStack.
Ref karma-runner#30
shirish87 added a commit to shirish87/karma-browserstack-launcher that referenced this issue Nov 5, 2015
Uses a shared poller to fetch and update the status of workers; helps reduce the no. of requests made to BrowserStack.
Ref karma-runner#30
marclaval pushed a commit to marclaval/karma-browserstack-launcher that referenced this issue Nov 5, 2015
Uses a shared poller to fetch and update the status of workers; helps reduce the no. of requests made to BrowserStack.
Ref karma-runner#30
@XhmikosR
Copy link
Contributor

XhmikosR commented Apr 9, 2020

@johnjbarton maybe you could change the default timeout to 10000 in this package and release a new version?

@johnjbarton
Copy link
Contributor

Did you try the suggestion in #159?

@XhmikosR
Copy link
Contributor

If you mean the suggestion in #159 (comment), I haven't tried that yet. I can try it but it's hard to reproduce unless I push many PRs at the same time.

So, just so that we are on the same page, the suggestion is to use this?

captureTimeout: 30000,
browserDisconnectTimeout: 30000,
browserDisconnectTolerance: 3
browserNoActivityTimeout: 30000,
browserSocketTimeout: 12000,

Someone else should try this values too. From #61 (comment) I see mentioning only these values:

captureTimeout: 30000,
browserDisconnectTimeout: 30000,
browserDisconnectTolerance: 3
browserNoActivityTimeout: 30000,

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

8 participants