Skip to content

Commit 4226261

Browse files
author
Horde
committed
Resync the vendored omni-github result schema
The copy under .github/actions/upload-omni-github-test-results was taken once in isaac-sim#6234 and never refreshed, so it has drifted from the source in NVIDIA-Omniverse/omni-github at docs/test-results/clients/artifact-upload/ schemas/result-json.schema.json. The upload action validates every result file against this copy and silently drops anything that fails, so a stale copy can reject payloads the service itself would accept. Replace it wholesale rather than patching individual rules. The file is now byte-identical to upstream. The drift is additive: upstream gained profileDetailString and seven optional testRow fields (error, failure_component, failure_layer, reliability_class, retry_kind, retry_reason, test_category). Nothing this repository relies on was removed. The converter's own output and the perf-smoke gate's result JSON both validate against the resynced schema unchanged. This also subsumes the oneOf -> anyOf fix in isaac-sim#6433, which upstream had already made; that PR remains the minimal standalone fix if this lands after it.
1 parent 48ec24a commit 4226261

1 file changed

Lines changed: 63 additions & 12 deletions

File tree

.github/actions/upload-omni-github-test-results/result-json.schema.json

Lines changed: 63 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -47,6 +47,11 @@
4747
"type": ["string", "null"],
4848
"maxLength": 4096
4949
},
50+
"profileDetailString": {
51+
"type": ["string", "null"],
52+
"minLength": 1,
53+
"maxLength": 1024
54+
},
5055
"artifactRelativePath": {
5156
"type": "string",
5257
"minLength": 1,
@@ -135,6 +140,10 @@
135140
"type": ["boolean", "null"],
136141
"default": false
137142
},
143+
"error": {
144+
"type": ["boolean", "null"],
145+
"default": false
146+
},
138147
"unreliable": {
139148
"type": ["boolean", "null"],
140149
"default": false
@@ -153,6 +162,47 @@
153162
"maximum": 100,
154163
"default": 0
155164
},
165+
"failure_layer": {
166+
"type": ["string", "null"],
167+
"enum": [
168+
"code",
169+
"infrastructure",
170+
"environment",
171+
"upstream",
172+
"downstream",
173+
null
174+
]
175+
},
176+
"failure_component": {
177+
"$ref": "#/$defs/profileDetailString"
178+
},
179+
"test_category": {
180+
"type": ["string", "null"],
181+
"enum": ["unit", "integration", "e2e", "smoke", null]
182+
},
183+
"reliability_class": {
184+
"type": ["string", "null"],
185+
"enum": [
186+
"stable",
187+
"flaky",
188+
"known-failure-rate",
189+
"experimental",
190+
null
191+
]
192+
},
193+
"retry_kind": {
194+
"type": ["string", "null"],
195+
"enum": [
196+
"infrastructure",
197+
"environment",
198+
"reliability_policy",
199+
"explicit",
200+
null
201+
]
202+
},
203+
"retry_reason": {
204+
"$ref": "#/$defs/profileDetailString"
205+
},
156206
"owner": {
157207
"$ref": "#/$defs/nullableShortString"
158208
},
@@ -241,23 +291,24 @@
241291
{
242292
"type": "array",
243293
"maxItems": 100,
244-
"items": {
245-
"anyOf": [
246-
{
294+
"anyOf": [
295+
{
296+
"items": {
247297
"type": "string",
248298
"maxLength": 1024
249-
},
250-
{
251-
"type": "integer"
252-
},
253-
{
299+
}
300+
},
301+
{
302+
"items": {
254303
"type": "number"
255-
},
256-
{
304+
}
305+
},
306+
{
307+
"items": {
257308
"type": "boolean"
258309
}
259-
]
260-
}
310+
}
311+
]
261312
},
262313
{
263314
"type": "object",

0 commit comments

Comments
 (0)