Merge pull request #3533 from artilleryio/fix/playwright-trace-upload

hassy · web-flow · commit 3d8b810a179b · 2025-05-09T09:55:33.000+01:00
feat: improve Playwright trace capture
diff --git a/packages/artillery-engine-playwright/index.js b/packages/artillery-engine-playwright/index.js
@@ -74,7 +74,7 @@ class PlaywrightEngine {
 
     // We use this to make sure only one VU is recording at one time:
     this.MAX_CONCURRENT_RECORDINGS =
-      this.tracingConfig.maxConcurrentRecordings || 3; // maximum number of VUs that can record at the same time
+      this.tracingConfig.maxConcurrentRecordings || 5; // maximum number of VUs that can record at the same time
     this.vusRecording = 0; // number of VUs currently recording
 
     //
diff --git a/packages/artillery/lib/platform/cloud/cloud.js b/packages/artillery/lib/platform/cloud/cloud.js
@@ -190,7 +190,7 @@ class ArtilleryCloudPlugin {
           );
         }
 
-        await this.waitOnUnprocessedLogs(30 * 1000); //just waiting for ee is not enough, as the api call takes time
+        await this.waitOnUnprocessedLogs(5 * 60 * 1000); //just waiting for ee is not enough, as the api call takes time
 
         if (isInteractiveUse) {
           await this._event('testrun:end', {

Original file line number	Diff line number	Diff line change
`@@ -190,7 +190,7 @@ class ArtilleryCloudPlugin {`
`190`	`190`	`);`
`191`	`191`	`}`
`192`	`192`
`193`		`- await this.waitOnUnprocessedLogs(30 * 1000); //just waiting for ee is not enough, as the api call takes time`
	`193`	`+ await this.waitOnUnprocessedLogs(5 * 60 * 1000); //just waiting for ee is not enough, as the api call takes time`
`194`	`194`
`195`	`195`	`if (isInteractiveUse) {`
`196`	`196`	`await this._event('testrun:end', {`