update vscdc push

Author: samruddhikhandale

.github/workflows/push-again.yml

@@ -53,6 +53,7 @@ jobs:
         REGISTRY_BASE_PATH: ${{ secrets.REGISTRY_BASE_PATH }}
         STUB_REGISTRY: ${{ secrets.STUB_REGISTRY }}
         STUB_REGISTRY_BASE_PATH: ${{ secrets.STUB_REGISTRY_BASE_PATH }}
+        VSCODE_REGISTRY_BASE_PATH: ${{ secrets.VSCODE_REGISTRY_BASE_PATH }}
       run: |
         set -e
 
@@ -77,4 +78,5 @@ jobs:
                           --registry-path "$REGISTRY_BASE_PATH" \
                           --stub-registry "$STUB_REGISTRY" \
                           --stub-registry-path "$STUB_REGISTRY_BASE_PATH" \
+                          --vscode-registry-path "$VSCODE_REGISTRY_BASE_PATH" \
                           ${{ github.event.inputs.image }}

.github/workflows/push-dev.yml

@@ -43,6 +43,7 @@ jobs:
         REGISTRY_BASE_PATH: ${{ secrets.REGISTRY_BASE_PATH }}
         STUB_REGISTRY: ${{ secrets.STUB_REGISTRY }}
         STUB_REGISTRY_BASE_PATH: ${{ secrets.STUB_REGISTRY_BASE_PATH }}
+        VSCODE_REGISTRY_BASE_PATH: ${{ secrets.VSCODE_REGISTRY_BASE_PATH }}
       run: |
         set -e
 
@@ -64,7 +65,8 @@ jobs:
                           --registry "$REGISTRY" \
                           --registry-path "$REGISTRY_BASE_PATH" \
                           --stub-registry "$STUB_REGISTRY" \
-                          --stub-registry-path "$STUB_REGISTRY_BASE_PATH"
+                          --stub-registry-path "$STUB_REGISTRY_BASE_PATH" \
+                          --vscode-registry-path "$VSCODE_REGISTRY_BASE_PATH"
 
   image-info:
     name: Trigger info extraction

.github/workflows/push.yml

@@ -47,6 +47,7 @@ jobs:
         REGISTRY_BASE_PATH: ${{ secrets.REGISTRY_BASE_PATH }}
         STUB_REGISTRY: ${{ secrets.STUB_REGISTRY }}
         STUB_REGISTRY_BASE_PATH: ${{ secrets.STUB_REGISTRY_BASE_PATH }}
+        VSCODE_REGISTRY_BASE_PATH: ${{ secrets.VSCODE_REGISTRY_BASE_PATH }}
       run: |
         set -e
 
@@ -64,7 +65,8 @@ jobs:
                           --registry "$REGISTRY" \
                           --registry-path "$REGISTRY_BASE_PATH" \
                           --stub-registry "$STUB_REGISTRY" \
-                          --stub-registry-path "$STUB_REGISTRY_BASE_PATH"
+                          --stub-registry-path "$STUB_REGISTRY_BASE_PATH" \
+                          --vscode-registry-path "$VSCODE_REGISTRY_BASE_PATH"
 
   image-info:
     name: Trigger info extraction

build/src/push.js

@@ -11,7 +11,7 @@ const prep = require('./prep');
 const builderName = 'dev-containers-builder';
 
 async function push(repo, release, updateLatest, registry, registryPath, stubRegistry,
-    stubRegistryPath, pushImages, prepOnly, definitionsToSkip, page, pageTotal, replaceImages, definitionId) {
+    stubRegistryPath, pushImages, prepOnly, definitionsToSkip, page, pageTotal, replaceImages, definitionId, vscodeRegistryPath) {
 
     // Optional argument defaults
     prepOnly = typeof prepOnly === 'undefined' ? false : prepOnly;
@@ -45,14 +45,14 @@ async function push(repo, release, updateLatest, registry, registryPath, stubReg
     await asyncUtils.forEach(definitionsToPush, async (currentDefinitionId) => {
         console.log(`**** Pushing ${currentDefinitionId} ${release} ****`);
         await pushImage(
-            currentDefinitionId, repo, release, updateLatest, registry, registryPath, stubRegistry, stubRegistryPath, prepOnly, pushImages, replaceImages);
+            currentDefinitionId, repo, release, updateLatest, registry, registryPath, stubRegistry, stubRegistryPath, prepOnly, pushImages, replaceImages, vscodeRegistryPath);
     });
 
     return stagingFolder;
 }
 
 async function pushImage(definitionId, repo, release, updateLatest,
-    registry, registryPath, stubRegistry, stubRegistryPath, prepOnly, pushImages, replaceImage) {
+    registry, registryPath, stubRegistry, stubRegistryPath, prepOnly, pushImages, replaceImage, vscodeRegistryPath) {
     const definitionPath = configUtils.getDefinitionPath(definitionId);
     const dotDevContainerPath = definitionPath;
     // Use Dockerfile for image build
@@ -93,9 +93,8 @@ async function pushImage(definitionId, repo, release, updateLatest,
             const imageNamesWithVersionTags = configUtils.getTagList(definitionId, release, updateLatest, registry, registryPath, variant);
             const imageName = imageNamesWithVersionTags[0].split(':')[0];
 
-            // Temporary change to dual publish an image to devcontainers and vscode/devcontainers
-            const vscodeContainerRegistyPath = configUtils.getConfig('vscodeContainerRegistryPath', 'public/vscode/devcontainers');
-            const vscodeImageNamesWithVersionTags = configUtils.getTagList(definitionId, release, updateLatest, registry, vscodeContainerRegistyPath, variant);
+            // Dual publish image to devcontainers and vscode/devcontainers
+            const vscodeImageNamesWithVersionTags = configUtils.getTagList(definitionId, release, updateLatest, registry, vscodeRegistryPath, variant);
             const vscodeImageName = vscodeImageNamesWithVersionTags[0].split(':')[0];
 
             console.log(`(*) Tags:${imageNamesWithVersionTags.reduce((prev, current) => prev += `\n     ${current}`, '')}`);

build/vscdc

@@ -81,6 +81,10 @@ require('yargs')
                     describe: 'A space separated list of image names to skip building and pushing.',
                     type: 'array',
                     default: []
+                },
+                'vscode-registry-path': {
+                    describe: 'VS Code container registry path',
+                    default: configUtils.getConfig('vscodeContainerRegistryPath', '')
                 }
             })
     }, pushCommand)
@@ -240,7 +244,7 @@ require('yargs')
 
 function pushCommand(argv) {
     push(argv.githubRepo, argv.release, argv.updateLatest, argv.registry, argv.registryPath, argv.stubRegistry,
-        argv.stubRegistryPath, argv.push, argv.prepOnly, argv.skip, argv.page, argv.pageTotal, argv.replaceImages, argv.devcontainer)
+        argv.stubRegistryPath, argv.push, argv.prepOnly, argv.skip, argv.page, argv.pageTotal, argv.replaceImages, argv.devcontainer, argv.vscodeRegistryPath)
         .catch((reason) => {
             console.error(`(!) Push failed - ${reason}`);
             if(reason.stack) {