feat: argus-docker-build allow skipping manifest update (#307)

chanzuckerberg · Sep 10, 2024 · a4e91c6 · a4e91c6
1 parent ecfb3b9
commit a4e91c6
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/.github/workflows/argus-docker-build.yaml b/.github/workflows/argus-docker-build.yaml
@@ -64,7 +64,8 @@ jobs:
             {
               "type": "object",
               "additionalProperties": {
-                type: "object",
+                "type": "object",
+                "additionalProperties": false,
                 "properties": {
                   "context": { "type": "string" },
                   "dockerfile": { "type": "string" },
@@ -95,6 +96,7 @@ jobs:
                     "type": "array",
                     "items": { "type": "string" }
                   },
+                  "skip_manifest_update": { "type": "boolean" }
                 },
                 "required": ["context", "dockerfile"]
               }
@@ -230,7 +232,7 @@ jobs:
           result-encoding: string
           script: |
             const images = ${{ needs.prep.outputs.images }};
-            const argusRootDirs = images.filter(image => image.should_build).map(image => image.argus_root);
+            const argusRootDirs = images.filter(image => image.should_build && !image.skip_manifest_update).map(image => image.argus_root);
             console.log('Argus root dirs:', argusRootDirs);
             return argusRootDirs.join(',');