Skip to content

Commit

Permalink
[newrelic-logging] Allow setting a custom metrics endpoint to bypass …
Browse files Browse the repository at this point in the history
…the broken autodetection (#1450)

#### Is this a new chart

No

#### What this PR does / why we need it:

Allows overriding the default metrics endpoint, as it's already possible
for the logs endpoint

#### Which issue this PR fixes
  - fixes #1399

#### Special notes for your reviewer:

#### Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove
unrelated fields.]
- [X] Chart Version bumped
- [ ] Variables are documented in the README.md
- [X] Title of the PR starts with chart name (e.g. `[mychartname]`)

---------

Co-authored-by: voorepreethi <[email protected]>
  • Loading branch information
giannello and voorepreethi authored Nov 7, 2024
1 parent b08cafe commit 79abdb3
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 1 deletion.
2 changes: 1 addition & 1 deletion charts/newrelic-logging/Chart.yaml
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
apiVersion: v2
description: A Helm chart to deploy New Relic Kubernetes Logging as a DaemonSet, supporting both Linux and Windows nodes and containers
name: newrelic-logging
version: 1.23.3
version: 1.23.4
appVersion: 2.1.0
home: https://github.com/newrelic/kubernetes-logging
icon: https://newrelic.com/assets/newrelic/source/NewRelic-logo-square.svg
Expand Down
2 changes: 2 additions & 0 deletions charts/newrelic-logging/templates/_helpers.tpl
Original file line number Diff line number Diff line change
Expand Up @@ -184,6 +184,8 @@ Returns metricsHost
{{- define "newrelic-logging.metricsHost" -}}
{{- if (include "newrelic.nrStaging" .) -}}
staging-metric-api.newrelic.com
{{- else if .Values.metricsEndpoint -}}
{{ .Values.metricsEndpoint -}}
{{- else if eq (substr 0 2 (include "newrelic-logging.licenseKey" .)) "eu" -}}
metric-api.eu.newrelic.com
{{- else -}}
Expand Down
1 change: 1 addition & 0 deletions charts/newrelic-logging/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -23,6 +23,7 @@
# whether your account is for the EU region or not.
#
# endpoint: https://log-api.newrelic.com/log/v1
# metricsEndpoint: metric-api.newrelic.com

fluentBit:
logLevel: "info"
Expand Down

0 comments on commit 79abdb3

Please sign in to comment.