Merge pull request #150 from crazy-max/check-img

check image exists before pulling

src/docker.ts

@@ -0,0 +1,15 @@
+import * as exec from '@actions/exec';
+
+export async function imageExists(name: string): Promise<boolean> {
+  return await exec
+    .getExecOutput('docker', ['images', '-q', name], {
+      ignoreReturnCode: true,
+      silent: true
+    })
+    .then(res => {
+      if (res.stderr.length > 0 && res.exitCode != 0) {
+        throw new Error(res.stderr);
+      }
+      return res.stdout.trim().length > 0;
+    });
+}

src/main.ts

@@ -4,6 +4,8 @@ import * as child_process from 'child_process';
 import * as core from '@actions/core';
 import * as exec from '@actions/exec';
 
+import * as docker from './docker';
+
 async function run() {
   try {
     if (os.platform() == 'darwin') {
@@ -22,31 +24,33 @@ async function run() {
       return;
     }
 
-    core.startGroup(`Pulling chocolatey Docker image`);
-    await exec.exec('docker', ['pull', image]);
-    core.endGroup();
-
-    core.startGroup('Running choco');
-    fs.writeFileSync('/tmp/env.txt', child_process.execSync(`env`, {encoding: 'utf8'}).trim());
-    await exec.exec('docker', [
-      'run',
-      '--rm',
-      '--env-file',
-      '/tmp/env.txt',
-      '--workdir',
-      '/wksp',
-      '--volume',
-      `${workspace}:/wksp`,
-      image,
-      args
-    ]);
-    core.endGroup();
-
-    core.startGroup('Fixing perms');
-    const uid = parseInt(child_process.execSync(`id -u`, {encoding: 'utf8'}).trim());
-    const gid = parseInt(child_process.execSync(`id -g`, {encoding: 'utf8'}).trim());
-    await exec.exec('sudo', ['chown', '-R', `${uid}:${gid}`, workspace]);
-    core.endGroup();
+    if (!(await docker.imageExists(image))) {
+      await core.group(`Pulling chocolatey Docker image`, async () => {
+        await exec.exec('docker', ['pull', image]);
+      });
+    }
+
+    await core.group(`Running choco`, async () => {
+      fs.writeFileSync('/tmp/env.txt', child_process.execSync(`env`, {encoding: 'utf8'}).trim());
+      await exec.exec('docker', [
+        'run',
+        '--rm',
+        '--env-file',
+        '/tmp/env.txt',
+        '--workdir',
+        '/wksp',
+        '--volume',
+        `${workspace}:/wksp`,
+        image,
+        args
+      ]);
+    });
+
+    await core.group(`Fixing perms`, async () => {
+      const uid = parseInt(child_process.execSync(`id -u`, {encoding: 'utf8'}).trim());
+      const gid = parseInt(child_process.execSync(`id -g`, {encoding: 'utf8'}).trim());
+      await exec.exec('sudo', ['chown', '-R', `${uid}:${gid}`, workspace]);
+    });
   } catch (error) {
     core.setFailed(error.message);
   }