Details
Description
The file indexing/secondary/docs/metrics_metadata.json has several errors - missing or extraneous commas which make it not parse as valid JSON.
More generally, now that the production Server build attempts to process these files as JSON rather than just copying them (CBD-5626), errors like this will break the Server build. Therefore we should add steps to validate these files at CMake time so developers will notice problems.
Attachments
For Gerrit Dashboard: MB-58826 | ||||||
---|---|---|---|---|---|---|
# | Subject | Branch | Project | Status | CR | V |
197708,1 | MB-58826: Fix JSON errors in metrics_metadata.json | unstable | indexing | Status: MERGED | +2 | +1 |
197717,2 | MB-58826: Introduce AddMetricsMetadata() | master | tlm | Status: MERGED | +2 | +1 |
197718,2 | MB-58826: Use AddMetricsMetadata() function | unstable | indexing | Status: MERGED | +2 | +1 |
197721,5 | MB-58826: Use AddMetricsMetadata() | trinity | cbas-core | Status: MERGED | +2 | +1 |
197722,2 | MB-58826: Use AddMetricsMetadata() | master | cbbs | Status: MERGED | +2 | +1 |
197723,3 | MB-58826: Use AddMetricsMetadata() | master | cbft | Status: MERGED | +2 | +1 |
197724,2 | MB-58826: Use AddMetricsMetadata() | unstable | eventing | Status: MERGED | +2 | +1 |
197725,2 | MB-58826: Use AddMetricsMetadata() | master | goxdcr | Status: MERGED | +2 | +1 |
197726,2 | MB-58826: Use AddMetricsMetadata() | master | query | Status: MERGED | +2 | +1 |
197727,2 | MB-58826: Use AddMetricsMetadata() | master | ns_server | Status: MERGED | +2 | +1 |
197728,2 | MB-58826: Use AddMetricsMetadata() | master | kv_engine | Status: MERGED | +2 | +1 |
197784,2 | MB-58826: Only require 'jq' for Linux production builds | master | tlm | Status: MERGED | +2 | +1 |
198241,2 | MB-58826: Use AddMetricsMetadata() | trinity | cbft | Status: MERGED | +2 | +1 |
198643,1 | Merge remote-tracking branch 'couchbase/trinity' into 'master' | master | cbft | Status: MERGED | +2 | +1 |