Skip to content

Commit e7f9852

Browse files
Ignore errors from private APIs (#3801) (#3803)
(cherry picked from commit 84571c3) Co-authored-by: Quentin Pradet <quentin.pradet@elastic.co>
1 parent 34c8786 commit e7f9852

File tree

2 files changed

+7
-60
lines changed

2 files changed

+7
-60
lines changed

compiler/src/steps/validate-model.ts

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -37,6 +37,8 @@ enum JsonEvent {
3737
array = 'array'
3838
}
3939

40+
const privateNamespaces = ['_internal', 'profiling']
41+
4042
/**
4143
* Validates the internal consistency of the model (doesn't check the json spec)
4244
*
@@ -208,6 +210,11 @@ export default async function validateModel (apiModel: model.Model, restSpec: Ma
208210
function validateEndpoint (endpoint: model.Endpoint): void {
209211
setRootContext(endpoint.name, 'request')
210212

213+
// Skip validation for internal endpoints
214+
if (privateNamespaces.some(ns => endpoint.name.startsWith(ns))) {
215+
return
216+
}
217+
211218
if (endpoint.request == null) {
212219
if (endpoint.response == null) {
213220
modelError('Missing request & response')

output/schema/validation-errors.json

Lines changed: 0 additions & 60 deletions
Original file line numberDiff line numberDiff line change
@@ -1,41 +1,5 @@
11
{
22
"endpointErrors": {
3-
"_internal.delete_desired_balance": {
4-
"request": [
5-
"Missing request & response"
6-
],
7-
"response": []
8-
},
9-
"_internal.delete_desired_nodes": {
10-
"request": [
11-
"Missing request & response"
12-
],
13-
"response": []
14-
},
15-
"_internal.get_desired_balance": {
16-
"request": [
17-
"Missing request & response"
18-
],
19-
"response": []
20-
},
21-
"_internal.get_desired_nodes": {
22-
"request": [
23-
"Missing request & response"
24-
],
25-
"response": []
26-
},
27-
"_internal.prevalidate_node_removal": {
28-
"request": [
29-
"Missing request & response"
30-
],
31-
"response": []
32-
},
33-
"_internal.update_desired_nodes": {
34-
"request": [
35-
"Missing request & response"
36-
],
37-
"response": []
38-
},
393
"async_search.submit": {
404
"request": [
415
"Request: query parameter 'min_compatible_shard_node' does not exist in the json spec",
@@ -471,30 +435,6 @@
471435
],
472436
"response": []
473437
},
474-
"profiling.flamegraph": {
475-
"request": [
476-
"Missing request & response"
477-
],
478-
"response": []
479-
},
480-
"profiling.stacktraces": {
481-
"request": [
482-
"Missing request & response"
483-
],
484-
"response": []
485-
},
486-
"profiling.status": {
487-
"request": [
488-
"Missing request & response"
489-
],
490-
"response": []
491-
},
492-
"profiling.topn_functions": {
493-
"request": [
494-
"Missing request & response"
495-
],
496-
"response": []
497-
},
498438
"reindex": {
499439
"request": [
500440
"Request: query parameter 'require_alias' does not exist in the json spec",

0 commit comments

Comments
 (0)