Merge pull request #72 from crazy-max/buildkit-version

Display BuildKit version
docker · Apr 28, 2021 · 0d135e0 · 0d135e0
2 parents 012185c + 36d8e00
commit 0d135e0
Show file tree

Hide file tree

Showing 5 changed files with 59 additions and 11 deletions.
diff --git a/.github/setup-buildx-action.png b/.github/setup-buildx-action.png
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -243,6 +243,8 @@ jobs:
         uses: ./
         with:
           endpoint: mycontext
+        env:
+          DOCKER_CONTEXT: mycontext
 
   config:
     runs-on: ubuntu-latest

diff --git a/dist/index.js b/dist/index.js
diff --git a/src/buildx.ts b/src/buildx.ts
@@ -20,7 +20,7 @@ export type Builder = {
 
 export async function getVersion(): Promise<string> {
   return await exec.exec(`docker`, ['buildx', 'version'], true).then(res => {
-    if (res.stderr != '' && !res.success) {
+    if (res.stderr.length > 0 && !res.success) {
       throw new Error(res.stderr);
     }
     return parseVersion(res.stdout);
@@ -37,7 +37,7 @@ export async function parseVersion(stdout: string): Promise<string> {
 
 export async function isAvailable(): Promise<Boolean> {
   return await exec.exec(`docker`, ['buildx'], true).then(res => {
-    if (res.stderr != '' && !res.success) {
+    if (res.stderr.length > 0 && !res.success) {
       return false;
     }
     return res.success;
@@ -46,7 +46,7 @@ export async function isAvailable(): Promise<Boolean> {
 
 export async function inspect(name: string): Promise<Builder> {
   return await exec.exec(`docker`, ['buildx', 'inspect', name], true).then(res => {
-    if (res.stderr != '' && !res.success) {
+    if (res.stderr.length > 0 && !res.success) {
       throw new Error(res.stderr);
     }
     const builder: Builder = {};
@@ -171,3 +171,21 @@ async function filename(version: string): Promise<string> {
   const ext: string = context.osPlat == 'win32' ? '.exe' : '';
   return util.format('buildx-v%s.%s-%s%s', version, platform, arch, ext);
 }
+
+export async function getBuildKitVersion(containerID: string): Promise<string> {
+  return exec.exec(`docker`, ['inspect', '--format', '{{.Config.Image}}', containerID], true).then(bkitimage => {
+    if (bkitimage.success && bkitimage.stdout.length > 0) {
+      return exec.exec(`docker`, ['run', '--rm', bkitimage.stdout, '--version'], true).then(bkitversion => {
+        if (bkitversion.success && bkitversion.stdout.length > 0) {
+          return `${bkitimage.stdout} => ${bkitversion.stdout}`;
+        } else if (bkitversion.stderr.length > 0) {
+          core.warning(bkitversion.stderr);
+        }
+        return bkitversion.stdout;
+      });
+    } else if (bkitimage.stderr.length > 0) {
+      core.warning(bkitimage.stderr);
+    }
+    return bkitimage.stdout;
+  });
+}
diff --git a/src/main.ts b/src/main.ts
@@ -81,6 +81,9 @@ async function run(): Promise<void> {
 
     if (inputs.driver == 'docker-container') {
       stateHelper.setContainerName(`buildx_buildkit_${builder.node_name}`);
+      core.startGroup(`BuildKit version`);
+      core.info(await buildx.getBuildKitVersion(`buildx_buildkit_${builder.node_name}`));
+      core.endGroup();
     }
     if (core.isDebug() || builder.node_flags?.includes('--debug')) {
       stateHelper.setDebug('true');
@@ -94,7 +97,7 @@ async function cleanup(): Promise<void> {
   if (stateHelper.IsDebug && stateHelper.containerName.length > 0) {
     core.startGroup(`BuildKit container logs`);
     await mexec.exec('docker', ['logs', `${stateHelper.containerName}`], false).then(res => {
-      if (res.stderr != '' && !res.success) {
+      if (res.stderr.length > 0 && !res.success) {
         core.warning(res.stderr);
       }
     });
@@ -104,7 +107,7 @@ async function cleanup(): Promise<void> {
   if (stateHelper.builderName.length > 0) {
     core.startGroup(`Removing builder`);
     await mexec.exec('docker', ['buildx', 'rm', `${stateHelper.builderName}`], false).then(res => {
-      if (res.stderr != '' && !res.success) {
+      if (res.stderr.length > 0 && !res.success) {
         core.warning(res.stderr);
       }
     });