Skip to content

Commit

Permalink
Tuning merge
Browse files Browse the repository at this point in the history
  • Loading branch information
wonko committed Feb 5, 2024
1 parent 8c97d52 commit 4778210
Show file tree
Hide file tree
Showing 3 changed files with 11 additions and 13 deletions.
4 changes: 2 additions & 2 deletions pkg/metricscollector/opentelemetry.go
Original file line number Diff line number Diff line change
Expand Up @@ -98,7 +98,7 @@ func initMeters() {
otLog.Error(err, msg)
}

otTriggerTotalsCounterDeprecated, err = meter.Int64UpDownCounter("keda.trigger.totals", api.WithDescription("DEPRECATED - will be removed in 2.15 - use 'keda.triggers.count' instead"))
otTriggerTotalsCounterDeprecated, err = meter.Int64UpDownCounter("keda.trigger.totals", api.WithDescription("DEPRECATED - will be removed in 2.16 - use 'keda.trigger.registered.count' instead"))
if err != nil {
otLog.Error(err, msg)
}
Expand All @@ -108,7 +108,7 @@ func initMeters() {
otLog.Error(err, msg)
}

otCrdTotalsCounterDeprecated, err = meter.Int64UpDownCounter("keda.resource.totals", api.WithDescription("DEPRECATED - will be removed in 2.15 - use 'keda.resources.count' instead"))
otCrdTotalsCounterDeprecated, err = meter.Int64UpDownCounter("keda.resource.totals", api.WithDescription("DEPRECATED - will be removed in 2.16 - use 'keda.resource.registered.count' instead"))
if err != nil {
otLog.Error(err, msg)
}
Expand Down
16 changes: 7 additions & 9 deletions pkg/metricscollector/prommetrics.go
Original file line number Diff line number Diff line change
Expand Up @@ -45,7 +45,7 @@ var (
Namespace: DefaultPromMetricsNamespace,
Subsystem: "scaler",
Name: "errors_total",
Help: "DEPRECATED - will be removed in 2.15 - use a `sum(scaler_errors_total{scaler!=\"\"})` over all scalers",
Help: "DEPRECATED - will be removed in 2.16 - use a `sum(scaler_errors_total{scaler!=\"\"})` over all scalers",
},
[]string{},
)
Expand All @@ -63,7 +63,7 @@ var (
Namespace: DefaultPromMetricsNamespace,
Subsystem: "scaler",
Name: "metrics_latency",
Help: "DEPRECATED - will be removed in 2.15 use 'scaler_metrics_latency_seconds' instead.",
Help: "DEPRECATED - will be removed in 2.16 use 'scaler_metrics_latency_seconds' instead.",
},
metricLabels,
)
Expand Down Expand Up @@ -99,7 +99,7 @@ var (
Namespace: DefaultPromMetricsNamespace,
Subsystem: "scaler",
Name: "errors",
Help: "DEPRECATED - will be removed in 2.15 - use 'scaler_errors_total' instead.",
Help: "DEPRECATED - will be removed in 2.16 - use 'scaler_errors_total' instead.",
},
metricLabels,
)
Expand All @@ -117,7 +117,7 @@ var (
Namespace: DefaultPromMetricsNamespace,
Subsystem: "scaled_object",
Name: "errors",
Help: "DEPRECATED - will be removed in 2.15 - use 'scaled_object_errors_total' instead.",
Help: "DEPRECATED - will be removed in 2.16 - use 'scaled_object_errors_total' instead.",
},
[]string{"namespace", "scaledObject"},
)
Expand Down Expand Up @@ -146,7 +146,7 @@ var (
Namespace: DefaultPromMetricsNamespace,
Subsystem: "trigger",
Name: "totals",
Help: "DEPRECATED - will be removed in 2.15 - use 'trigger_registered_total' instead.",
Help: "DEPRECATED - will be removed in 2.16 - use 'trigger_registered_total' instead.",
},
[]string{"type"},
)
Expand All @@ -164,7 +164,7 @@ var (
Namespace: DefaultPromMetricsNamespace,
Subsystem: "resource",
Name: "totals",
Help: "DEPRECATED - will be removed in 2.15 - use 'resource_handled_total' instead.",
Help: "DEPRECATED - will be removed in 2.16 - use 'resource_handled_total' instead.",
},
[]string{"type", "namespace"},
)
Expand All @@ -182,7 +182,7 @@ var (
Namespace: DefaultPromMetricsNamespace,
Subsystem: "internal_scale_loop",
Name: "latency",
Help: "DEPRECATED - will be removed in 2.15 - use 'internal_scale_loop_latency_seconds' instead.",
Help: "DEPRECATED - will be removed in 2.16 - use 'internal_scale_loop_latency_seconds' instead.",
},
[]string{"namespace", "type", "resource"},
)
Expand Down Expand Up @@ -240,8 +240,6 @@ func NewPromMetrics() *PromMetrics {
metrics.Registry.MustRegister(crdRegistered)
metrics.Registry.MustRegister(scaledJobErrors)

// metrics.Registry.MustRegister(triggerTotalsGaugeVec)
// metrics.Registry.MustRegister(crdTotalsGaugeVec)
metrics.Registry.MustRegister(buildInfo)

metrics.Registry.MustRegister(cloudeventEmitted)
Expand Down
4 changes: 2 additions & 2 deletions pkg/metricscollector/webhook/webhook_prommetrics.go
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,7 @@ var (
Namespace: DefaultPromMetricsNamespace,
Subsystem: "webhook",
Name: "scaled_object_validation_total",
Help: "DEPRECATED - will be removed in 2.15 - Use `scaled_object_validations_total` instead.",
Help: "DEPRECATED - will be removed in 2.16 - Use `scaled_object_validations_total` instead.",
},
[]string{"namespace", "action"},
)
Expand All @@ -49,7 +49,7 @@ var (
Namespace: DefaultPromMetricsNamespace,
Subsystem: "webhook",
Name: "scaled_object_validation_errors",
Help: "DEPRECATED - will be removed in 2.15 - Use `scaled_object_validation_errors_total` instead.",
Help: "DEPRECATED - will be removed in 2.16 - Use `scaled_object_validation_errors_total` instead.",
},
[]string{"namespace", "action", "reason"},
)
Expand Down

0 comments on commit 4778210

Please sign in to comment.