fix: run webpagereplay using the GUI adding --webpagereplay

sitespeedio · Jul 29, 2024 · e80b9e1 · e80b9e1
1 parent 11eed39
commit e80b9e1
Showing 1 changed file with 7 additions and 7 deletions.
diff --git a/testrunner/src/testrunners/docker-testrunner.js b/testrunner/src/testrunners/docker-testrunner.js
@@ -52,12 +52,12 @@ export default async function runJob(job) {
     );
 
     const parameters = setupDockerParameters(
-      job,
       dockerContainer,
       dockerExtraParameters,
       workingDirectory,
       configFileName,
-      resultFileName
+      resultFileName,
+      job.data.extras.includes('--webpagereplay')
     );
 
     if (job.data.scripting) {
@@ -150,12 +150,12 @@ async function handleScriptingFile(job, workingDirectory) {
 }
 
 function setupDockerParameters(
-  job,
   dockerContainer,
   dockerExtraParameters,
   workingDirectory,
   configFileName,
-  resultFileName
+  resultFileName,
+  usingWebPageReplay
 ) {
   const baseParameters = [
     'run',
@@ -172,9 +172,9 @@ function setupDockerParameters(
     true
   ];
 
-  if (job.data.config.webpagereplay) {
-    baseParameters.splice(1, 0, '--cap-add=NET_ADMIN');
-    baseParameters.splice(2, 0, '-e', 'REPLAY=true', '-e', 'LATENCY=100');
+  //
+  if (usingWebPageReplay) {
+    baseParameters.splice(1, 0, '-e', 'REPLAY=true', '-e', 'LATENCY=100');
   }
   return baseParameters;
 }