Skip to content

Commit

Permalink
Update schema
Browse files Browse the repository at this point in the history
  • Loading branch information
jefchien committed Jun 17, 2024
1 parent 778c028 commit 296c249
Show file tree
Hide file tree
Showing 3 changed files with 23 additions and 31 deletions.
8 changes: 3 additions & 5 deletions translator/config/sampleSchema/invalidLogsWithJMX.json
Original file line number Diff line number Diff line change
@@ -1,11 +1,9 @@
{
"logs": {
"metrics_collected": {
"application_signals": {
"jmx": {
"jvm": {
"measurement": []
}
"jmx": {
"jvm": {
"measurement": []
}
}
}
Expand Down
36 changes: 15 additions & 21 deletions translator/config/sampleSchema/validLogsWithJMX.json
Original file line number Diff line number Diff line change
Expand Up @@ -2,30 +2,24 @@
"logs": {
"metrics_collected": {
"application_signals": {
"jmx": {
"jvm": {
"measurement": [
"jvm.classes.loaded"
]
}
"enable_jvm": true
},
"jmx": {
"jvm": {
"measurement": [
"jvm.classes.loaded"
]
},
"tomcat": {
"measurement": [
{
"name": "tomcat.sessions",
"rename": "TOMCAT_SESSIONS"
}
]
}
},
"kubernetes": {
"jmx": {
"jvm": {
"measurement": [
"jvm.classes.loaded"
]
},
"tomcat": {
"measurement": [
{
"name": "tomcat.sessions",
"rename": "TOMCAT_SESSIONS"
}
]
}
}
}
}
}
Expand Down
10 changes: 5 additions & 5 deletions translator/config/schema.json
Original file line number Diff line number Diff line change
Expand Up @@ -721,15 +721,18 @@
]
}
},
"jmx": {
"$ref": "#/definitions/jmxTargetsDefinition"
"enable_jvm": {
"type": "boolean"
}
},
"tls": {
"$ref": "#/definitions/metricsDefinition/definitions/tlsDefinitions"
},
"additionalProperties": true
},
"jmx": {
"$ref": "#/definitions/jmxTargetsDefinition"
},
"ecs": {
"type": "object",
"properties": {
Expand Down Expand Up @@ -757,9 +760,6 @@
"disable_metric_extraction": {
"description": "Disable the extraction of metrics from EMF logs",
"type": "boolean"
},
"jmx": {
"$ref": "#/definitions/jmxTargetsDefinition"
}
},
"additionalProperties": true
Expand Down

0 comments on commit 296c249

Please sign in to comment.