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 pedro-stanaka committed Jan 18, 2023
1 parent a764aef commit 2f0d51b
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 @@ -43,6 +43,7 @@ To learn more about our roadmap, we recommend reading [this document](ROADMAP.md
- **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 @@ -72,7 +72,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 2f0d51b

Please sign in to comment.