Skip to content

Commit

Permalink
Replace backend latency aggregation function sum by mean (#470)
Browse files Browse the repository at this point in the history
* Replace backend latency aggregation function sum by mean

* Replace backend latency aggregation function sum by mean in yaml files

---------

Co-authored-by: JMHAUTBOIS-Claranet <[email protected]>
  • Loading branch information
BenjyOC and JMHAUTBOIS-Claranet authored Apr 18, 2023
1 parent a35e996 commit 6a9cf6b
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 4 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@ name: backend latency per service
id: backend_latency_service

transformation: true
aggregation: ".sum(by=['forwarding_rule_name', 'backend_target_name'])"
aggregation: ".mean(by=['forwarding_rule_name', 'backend_target_name'])"
filtering: "filter('service', 'loadbalancing')"
value_unit: "Millisecond"

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@ name: backend latency per bucket
id: backend_latency_bucket

transformation: true
aggregation: ".sum(by=['forwarding_rule_name', 'backend_target_name'])"
aggregation: ".mean(by=['forwarding_rule_name', 'backend_target_name'])"
filtering: "filter('service', 'loadbalancing')"
value_unit: "Millisecond"

Expand Down
4 changes: 2 additions & 2 deletions modules/integration_gcp-load-balancing/variables-gen.tf
Original file line number Diff line number Diff line change
Expand Up @@ -212,7 +212,7 @@ variable "backend_latency_service_notifications" {
variable "backend_latency_service_aggregation_function" {
description = "Aggregation function and group by for backend_latency_service detector (i.e. \".mean(by=['host'])\")"
type = string
default = ".sum(by=['forwarding_rule_name', 'backend_target_name'])"
default = ".mean(by=['forwarding_rule_name', 'backend_target_name'])"
}

variable "backend_latency_service_transformation_function" {
Expand Down Expand Up @@ -302,7 +302,7 @@ variable "backend_latency_bucket_notifications" {
variable "backend_latency_bucket_aggregation_function" {
description = "Aggregation function and group by for backend_latency_bucket detector (i.e. \".mean(by=['host'])\")"
type = string
default = ".sum(by=['forwarding_rule_name', 'backend_target_name'])"
default = ".mean(by=['forwarding_rule_name', 'backend_target_name'])"
}

variable "backend_latency_bucket_transformation_function" {
Expand Down

0 comments on commit 6a9cf6b

Please sign in to comment.