Skip to content

Commit 9bc6700

Browse files
author
Nick Metz
committed
Make field optional since if aggregation is doc_count it's should not be set
Signed-off-by: Nick Metz <[email protected]>
1 parent 5f2154e commit 9bc6700

File tree

3 files changed

+340
-81
lines changed

3 files changed

+340
-81
lines changed

docs/resources/kibana_slo.md

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -399,13 +399,13 @@ Required:
399399

400400
Required:
401401

402-
- `aggregation` (String)
403-
- `field` (String)
404-
- `name` (String)
402+
- `aggregation` (String) The aggregation type for this metric. One of: sum, doc_count. Determines which other fields are required:
403+
- `name` (String) The unique name for this metric. Used as a variable in the equation field.
405404

406405
Optional:
407406

408-
- `filter` (String)
407+
- `field` (String) Field to aggregate. Must NOT be set for doc_count.
408+
- `filter` (String) Optional KQL filter for this metric.
409409

410410

411411

@@ -422,13 +422,13 @@ Required:
422422

423423
Required:
424424

425-
- `aggregation` (String)
426-
- `field` (String)
427-
- `name` (String)
425+
- `aggregation` (String) The aggregation type for this metric. One of: sum, doc_count. Determines which other fields are required:
426+
- `name` (String) The unique name for this metric. Used as a variable in the equation field.
428427

429428
Optional:
430429

431-
- `filter` (String)
430+
- `field` (String) Field to aggregate. Must NOT be set for doc_count.
431+
- `filter` (String) Optional KQL filter for this metric.
432432

433433

434434

0 commit comments

Comments
 (0)