Skip to content

Commit

Permalink
fix: tools-2772 validate logging context drv-mem (#26)
Browse files Browse the repository at this point in the history
* fix: validate logging context drv-mem using generated server7 schema

* chore: update to operator team server 7 schema

* ci: update server7 test with logging context drv-mem

* ci: fix tests for server7 schema storage-engine oneof re-order
  • Loading branch information
dwelch-spike authored Dec 6, 2023
1 parent 29b7ac6 commit f241a18
Show file tree
Hide file tree
Showing 4 changed files with 9 additions and 3 deletions.
4 changes: 2 additions & 2 deletions integration_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -1009,7 +1009,7 @@ var validateTests = []validateTest{
expectedResult: `context: (root).namespaces.0
- description: Additional property memory-size is not allowed, error-type: additional_property_not_allowed
context: (root).namespaces.0.storage-engine
- description: data-size is required, error-type: required
- description: devices is required, error-type: required
context: (root).namespaces.1
- description: Additional property memory-size is not allowed, error-type: additional_property_not_allowed
context: (root).namespaces.1.index-type
Expand All @@ -1021,7 +1021,7 @@ context: (root).namespaces.1.sindex-type
- description: Additional property mounts-size-limit is not allowed, error-type: additional_property_not_allowed
- description: mounts-budget is required, error-type: required
context: (root).namespaces.1.storage-engine
- description: data-size is required, error-type: required
- description: devices is required, error-type: required
context: (root).service
- description: cluster-name is required, error-type: required
`,
Expand Down
2 changes: 1 addition & 1 deletion schema/schemas
Submodule schemas updated 1 files
+2,626 −2,620 json/aerospike/7.0.0.json
4 changes: 4 additions & 0 deletions testdata/cases/server70/server70.conf
Original file line number Diff line number Diff line change
Expand Up @@ -21,6 +21,10 @@ logging {
file /dummy/file/path2 {
context any info
}

console {
context drv-mem debug
}
}

network {
Expand Down
2 changes: 2 additions & 0 deletions testdata/cases/server70/server70.yaml
Original file line number Diff line number Diff line change
@@ -1,4 +1,6 @@
logging:
- drv-mem: debug
name: console
- any: info
name: /dummy/file/path2
namespaces:
Expand Down

0 comments on commit f241a18

Please sign in to comment.