Add Docker image publishing to packaging scripts

api3dao · Oct 27, 2022 · a52476a · a52476a
1 parent 31c9945
commit a52476a
Show file tree

Hide file tree

Showing 4 changed files with 100 additions and 54 deletions.
diff --git a/.github/workflows/build-test.yml b/.github/workflows/build-test.yml
@@ -129,20 +129,14 @@ jobs:
       - name: Publish NPM packages to local NPM registry
         run: yarn docker:scripts:publish-packages --npm-registry local --npm-tag ${{ github.sha }} --snapshot
       - name: Build Docker containers
-        run: yarn docker:scripts:build-docker-images --dev --npm-registry local --npm-tag ${{ github.sha }} --docker-tag ${{ github.sha }}
+        run: yarn docker:scripts:docker:build --dev --npm-registry local --npm-tag ${{ github.sha }} --docker-tag ${{ github.sha }}
       - name: Stop local NPM registry
         run: yarn docker:scripts:npm-registry:stop
-      - name: Login to DockerHub
-        uses: docker/login-action@v2
-        with:
-          username: ${{ secrets.DOCKERHUB_USERNAME }}
-          password: ${{ secrets.DOCKERHUB_TOKEN }}
-      - name: Push airnode-admin to Docker Hub
-        run: docker push api3/airnode-admin-dev:${{ github.sha }}
-      - name: Push airnode-client to Docker Hub
-        run: docker push api3/airnode-client-dev:${{ github.sha }}
-      - name: Push airnode-deployer to Docker Hub
-        run: docker push api3/airnode-deployer-dev:${{ github.sha }}
+      - name: Publish Docker containers
+        env:
+          DOCKERHUB_USERNAME: ${{ secrets.DOCKERHUB_USERNAME }}
+          DOCKERHUB_TOKEN: ${{ secrets.DOCKERHUB_TOKEN }}
+        run: yarn docker:scripts:docker:publish --dev --docker-tag ${{ github.sha }}
   unit-tests:
     name: Unit tests
     runs-on: ubuntu-latest

diff --git a/docker/scripts/cli.ts b/docker/scripts/cli.ts
@@ -3,7 +3,7 @@ import omitBy from 'lodash/omitBy';
 import { logger } from '@api3/airnode-utilities';
 import { go, GoResult, goSync } from '@api3/promise-utils';
 import { stopNpmRegistry, startNpmRegistry } from './npm-registry';
-import { buildDockerImages } from './build-docker-images';
+import { buildDockerImages, publishDockerImages } from './docker';
 import { publishPackages } from './publish-packages';
 
 // Taken from airnode-deployer
@@ -88,42 +88,72 @@ yargs(process.argv.slice(2))
       });
     }
   )
-  .command(
-    'build-docker-images',
-    'Build Docker images',
-    {
-      'npm-registry': {
-        alias: 'r',
-        description: 'NPM registry URL to fetch packages from or a keyword `local` to use a local NPM registry',
-        default: 'https://registry.npmjs.org/',
-        type: 'string',
-      },
-      'npm-tag': {
-        alias: 't',
-        description: 'NPM tag/version of the packages that will be fetched',
-        default: 'latest',
-        type: 'string',
-      },
-      'docker-tag': {
-        alias: 'g',
-        description: 'Docker tag to build the images under',
-        default: 'latest',
-        type: 'string',
-      },
-      dev: {
-        alias: 'd',
-        description: 'Build Docker dev images (with -dev suffix)',
-        default: false,
-        type: 'boolean',
-      },
-    },
-    (args) => {
-      logger.log(`Running command '${args._[0]}' with arguments ${longArguments(args)}`);
-      runCliCommand(() => {
-        buildDockerImages(args.npmRegistry, args.npmTag, args.dockerTag, args.dev);
-      });
-    }
-  )
+  .command('docker', 'Manages Docker images', (yargs) => {
+    yargs
+      .command(
+        'build',
+        'Build Docker images',
+        {
+          'npm-registry': {
+            alias: 'r',
+            description: 'NPM registry URL to fetch packages from or a keyword `local` to use a local NPM registry',
+            default: 'https://registry.npmjs.org/',
+            type: 'string',
+          },
+          'npm-tag': {
+            alias: 't',
+            description: 'NPM tag/version of the packages that will be fetched',
+            default: 'latest',
+            type: 'string',
+          },
+          'docker-tag': {
+            alias: 'g',
+            description: 'Docker tag to build the images under',
+            default: 'latest',
+            type: 'string',
+          },
+          dev: {
+            alias: 'd',
+            description: 'Build Docker dev images (with -dev suffix)',
+            default: false,
+            type: 'boolean',
+          },
+        },
+        (args) => {
+          logger.log(`Running command '${args._[0]} ${args._[1]}' with arguments ${longArguments(args)}`);
+          runCliCommand(() => {
+            buildDockerImages(args.npmRegistry, args.npmTag, args.dockerTag, args.dev);
+          });
+        }
+      )
+      .command(
+        'publish',
+        'Publish Docker images',
+        {
+          'docker-tag': {
+            alias: 'g',
+            description: 'Docker tag to build the images under',
+            default: 'latest',
+            type: 'string',
+          },
+          dev: {
+            alias: 'd',
+            description: 'Build Docker dev images (with -dev suffix)',
+            default: false,
+            type: 'boolean',
+          },
+        },
+        (args) => {
+          logger.log(`Running command '${args._[0]} ${args._[1]}' with arguments ${longArguments(args)}`);
+          runCliCommand(() => {
+            publishDockerImages(args.dockerTag, args.dev);
+          });
+        }
+      )
+      .help()
+      .demandCommand(1)
+      .strict();
+  })
   .help()
   .demandCommand(1)
   .strict()

diff --git a/docker/scripts/build-docker-images.ts → docker/scripts/docker.ts b/docker/scripts/build-docker-images.ts → docker/scripts/docker.ts
@@ -26,3 +26,23 @@ export const buildDockerImages = (npmRegistry: string, npmTag: string, dockerTag
     `docker build --no-cache --build-arg npmRegistryUrl=${npmRegistryUrl} --build-arg npmTag=${npmTag} --tag api3/airnode-client${devSuffix}:${dockerTag} --file /app/airnode-client/Dockerfile /app/airnode-client`
   );
 };
+
+const loginDockerHub = () => {
+  const username = process.env.DOCKERHUB_USERNAME;
+  const password = process.env.DOCKERHUB_TOKEN;
+
+  if (!username || !password) {
+    throw new Error('Missing DockerHub credentials');
+  }
+
+  runCommand(`docker login --password-stdin --username ${username}`, { input: password });
+};
+
+export const publishDockerImages = (dockerTag: string, dev: boolean) => {
+  const devSuffix = dev ? '-dev' : '';
+
+  loginDockerHub();
+  runCommand(`docker push api3/airnode-admin${devSuffix}:${dockerTag}`);
+  runCommand(`docker push api3/airnode-deployer${devSuffix}:${dockerTag}`);
+  runCommand(`docker push api3/airnode-client${devSuffix}:${dockerTag}`);
+};
diff --git a/package.json b/package.json
@@ -45,15 +45,17 @@
     "dev:invoke": "(cd packages/airnode-node && yarn run dev:invoke)",
     "dev:list": "(cd packages/airnode-operation && yarn run dev:list)",
     "dev:stop": "(cd packages/airnode-operation && yarn run dev:stop)",
-    "docker:build:images:latest": "yarn docker:scripts:build-docker-images:latest",
-    "docker:build:images:local": "yarn docker:scripts:npm-registry:start && yarn docker:scripts:publish-packages:local && yarn docker:scripts:build-docker-images:local && yarn docker:scripts:npm-registry:stop",
+    "docker:build:images:latest": "yarn docker:scripts:docker:build:latest",
+    "docker:build:images:local": "yarn docker:scripts:npm-registry:start && yarn docker:scripts:publish-packages:local && yarn docker:scripts:docker:build:local && yarn docker:scripts:npm-registry:stop",
     "docker:build:latest": "yarn docker:build:packaging && yarn docker:build:images:latest",
     "docker:build:local": "yarn docker:build:packaging && yarn docker:build:images:local",
     "docker:build:packaging": "yarn docker:scripts:build && docker build --tag api3/airnode-packaging:latest --file docker/Dockerfile .",
     "docker:scripts:build": "ncc build docker/scripts/cli.ts -o docker/scripts/dist --no-cache --minify --source-map --transpile-only",
-    "docker:scripts:build-docker-images": "docker run --rm -v /var/run/docker.sock:/var/run/docker.sock api3/airnode-packaging:latest build-docker-images",
-    "docker:scripts:build-docker-images:latest": "yarn docker:scripts:build-docker-images",
-    "docker:scripts:build-docker-images:local": "yarn docker:scripts:build-docker-images --npm-registry local --npm-tag local --docker-tag local",
+    "docker:scripts:docker:build": "docker run --rm -v /var/run/docker.sock:/var/run/docker.sock api3/airnode-packaging:latest docker build",
+    "docker:scripts:docker:build:latest": "yarn docker:scripts:docker:build",
+    "docker:scripts:docker:build:local": "yarn docker:scripts:docker:build --npm-registry local --npm-tag local --docker-tag local",
+    "docker:scripts:docker:publish": "docker run --rm -v /var/run/docker.sock:/var/run/docker.sock -e DOCKERHUB_USERNAME -e DOCKERHUB_TOKEN api3/airnode-packaging:latest docker publish",
+    "docker:scripts:docker:publish:latest": "yarn docker:scripts:docker:publish",
     "docker:scripts:npm-registry": "docker run --rm -v /var/run/docker.sock:/var/run/docker.sock api3/airnode-packaging:latest npm-registry",
     "docker:scripts:npm-registry:start": "yarn docker:scripts:npm-registry start",
     "docker:scripts:npm-registry:stop": "yarn docker:scripts:npm-registry stop",