Fix ce-release-artifacts.json

Add gate test to ensure this file remains json
alaric27 · Aug 1, 2023 · d673bd6 · d673bd6
1 parent e8bf539
commit d673bd6
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 3 deletions.
diff --git a/graal-common.json b/graal-common.json
@@ -1,6 +1,6 @@
 {
   "README": "This file contains definitions that are useful for the jsonnet CI files of the graal and graal-enterprise repositories.",
   "ci": {
-    "overlay": "32dabcc2b777b4bbd934f634b3a07ad5ce4e8725"
+    "overlay": "caeb7fc170b36ed926bef9a9b4aefe82779db376"
   }
 }
diff --git a/vm/ce-release-artifacts.json b/vm/ce-release-artifacts.json
@@ -316,7 +316,7 @@
             },
             {
                 "os": "windows",
-                "arch": "amd64",
+                "arch": "amd64"
             }
         ]
     },
@@ -343,7 +343,7 @@
             },
             {
                 "os": "windows",
-                "arch": "amd64",
+                "arch": "amd64"
             }
         ]
     },

diff --git a/vm/mx.vm/mx_vm_gate.py b/vm/mx.vm/mx_vm_gate.py
@@ -24,6 +24,7 @@
 # questions.
 #
 # ----------------------------------------------------------------------------------------------------
+import json
 import shutil
 
 import mx
@@ -448,6 +449,12 @@ def gate_body(args, tasks):
         if t:
             mx_sdk_vm.verify_graalvm_configs(suites=['vm', 'vm-enterprise'])
 
+    with Task('Vm: ce-release-artifacts.json', tasks, tags=['style']) as t:
+        if t:
+            with open(join(_suite.dir, 'ce-release-artifacts.json'), 'r') as f:
+                # check that this file can be read as json
+                json.load(f)
+
     with Task('Vm: Basic GraalVM Tests', tasks, tags=[VmGateTasks.compiler]) as t:
         if t and mx_sdk_vm_impl.has_component('GraalVM compiler'):
             # 1. the build must be a GraalVM