allow passing multiple device options to docker (#1655)

Author: djpnewton

src/argv.ts

@@ -215,10 +215,9 @@ export class Argv {
         return this.map.get("privileged") ?? false;
     }
 
-    get device (): string | null {
-        const device = this.map.get("device");
-        if (!device) return null;
-        return device;
+    get device (): string[] {
+        const val = this.map.get("device") ?? [];
+        return typeof val == "string" ? val.split(" ") : val;
     }
 
     get ulimit (): string | null {

src/index.ts

@@ -218,8 +218,8 @@ process.on("SIGUSR2", async () => await cleanupJobResources(jobs));
             requiresArg: false,
         })
         .option("device", {
-            type: "string",
-            description: "Set docker executor device option",
+            type: "array",
+            description: "Add devices to docker executor",
             requiresArg: false,
         })
         .option("ulimit", {

src/job.ts

@@ -837,8 +837,8 @@ export class Job {
                 dockerCmd += "--privileged ";
             }
 
-            if (this.argv.device) {
-                dockerCmd += `--device=${this.argv.device} `;
+            for (const device of this.argv.device) {
+                dockerCmd += `--device ${device} `;
             }
 
             if (this.argv.ulimit !== null) {
@@ -1418,8 +1418,8 @@ export class Job {
             dockerCmd += "--privileged ";
         }
 
-        if (this.argv.device) {
-            dockerCmd += `--device=${this.argv.device} `;
+        for (const device of this.argv.device) {
+            dockerCmd += `--device ${device} `;
         }
 
         if (this.argv.ulimit !== null) {