Skip to content

Commit

Permalink
NewRelic scaler crashes on logging (kedacore#3946)
Browse files Browse the repository at this point in the history
Signed-off-by: Laszlo Kishalmi <laszlo.kishalmi@partech.com>

Signed-off-by: Laszlo Kishalmi <laszlo.kishalmi@partech.com>
  • Loading branch information
lkishalmi authored and josephangbc committed Dec 6, 2022
1 parent 5901daa commit 13c273b
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 1 deletion.
1 change: 1 addition & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -77,6 +77,7 @@ To learn more about active deprecations, we recommend checking [GitHub Discussio
- **General:** Provide patch for CVE-2022-3172 vulnerability ([#3690](https://github.com/kedacore/keda/issues/3690))
- **General:** Respect optional parameter inside envs for ScaledJobs ([#3568](https://github.com/kedacore/keda/issues/3568))
- **Azure Blob Scaler** Store forgotten logger ([#3811](https://github.com/kedacore/keda/issues/3811))
- **New Relic Scaler** Store forgotten logger ([#3945](https://github.com/kedacore/keda/issues/3945))
- **Prometheus Scaler:** Treat Inf the same as Null result ([#3644](https://github.com/kedacore/keda/issues/3644))
- **NATS Jetstream:** Correctly count messages that should be redelivered (waiting for ack) towards keda value ([#3787](https://github.com/kedacore/keda/issues/3787))

Expand Down
3 changes: 2 additions & 1 deletion pkg/scalers/newrelic_scaler.go
Original file line number Diff line number Diff line change
Expand Up @@ -71,7 +71,8 @@ func NewNewRelicScaler(config *ScalerConfig) (Scaler, error) {
return &newrelicScaler{
metricType: metricType,
metadata: meta,
nrClient: nrClient}, nil
nrClient: nrClient,
logger: logger}, nil
}

func parseNewRelicMetadata(config *ScalerConfig, logger logr.Logger) (*newrelicMetadata, error) {
Expand Down

0 comments on commit 13c273b

Please sign in to comment.