-
Notifications
You must be signed in to change notification settings - Fork 7.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
fix(core): Fix telemetry for concurrency control #9845
Conversation
Capacity is a negative integer when there are queued items, but reportable thresholds are positive.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM - maybe one thing we could do is set a soft of debounce to avoid flooding telemetry just as precaution
4 flaky tests on run #5687 ↗︎
Details:
5-ndv.cy.ts • 2 flaky tests
10-undo-redo.cy.ts • 2 flaky tests
Review all test suite changes for PR #9845 ↗︎ |
✅ All Cypress E2E specs passed |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🙇🏽
✅ All Cypress E2E specs passed |
Got released with |
Concurrency limit for production executions on regular mode is currently set on cloud to
999
so that we can gather data in order to set reasonable limits for cloud plans.On telemetry we had a misunderstanding on what needs to be reported - we are now reporting as follows:
capacity
becomes994
, reportUser hit concurrency limit {threshold: 5}
capacity
becomes989
, reportUser hit concurrency limit {threshold: 10}
capacity
becomes979
, reportUser hit concurrency limit {threshold: 20}
capacity
becomes949
, reportUser hit concurrency limit {threshold: 50}
capacity
becomes899
, reportUser hit concurrency limit {threshold: 100}
capacity
becomes799
, reportUser hit concurrency limit {threshold: 200}
Bear in mind this cloud-specific telemetry setup is temporary until data is gathered for a few weeks. Once cloud plan limits are set, this telemetry setup will be removed from concurrency control.
https://linear.app/n8n/issue/PAY-1627/concurrency-control-for-main-mode