Commit 9255d17
authored
[newrelic-logging] support GOV (FedRAMP) endpoints via common-library 2.3.3 (#2320)
#### What this PR does / why we need it:
Enables New Relic Kubernetes Logging to target GOV (FedRAMP) endpoints.
Consumer-side follow-up to
#[2318](#2318), which added
the GOV endpoints to common-library.
- Bump `common-library` dependency 2.3.1 → 2.3.3 (Chart.yaml +
Chart.lock)
- Bump chart version 1.38.2 → 1.38.3
- Add `tests/endpoint_region_selection_test.yaml` covering region →
endpoint resolution (staging / US / EU / GOV / custom / defaults)
When `global.fedramp.enabled=true`, endpoints resolve to:
- Log API → `https://gov-log-api.newrelic.com/log/v1`
- Metric host → `gov-metric-api.newrelic.com
## Verification
update on the GOV endpoint test
Deployed the chart with --set global.fedramp.enabled=true and confirmed
it picks the GOV endpoint — DaemonSet env
ENDPOINT=https://gov-log-api.newrelic.com/log/v1, and the plugin logs
the same URL it dials at runtime.
Sent logs with a US ingestion key → they landed in
[NRDB](https://onenr.io/0gR7POl8xjo), but in a regular US account
(756053). Reason: routing to an account/cell is decided by the license
key, not the endpoint. The gov endpoint just authenticates the key and
routes to that key's account — so a US key lands in a US cell even via
the gov endpoint; a fed-account key would route to a fed cell.
So:
- fedramp.enabled=true correctly targets the GOV endpoint and the full
ingest pipeline works
- Data landed in a US cell because the key is a US key — this doesn't
confirm fed-cell ingestion
#### Which issue this PR fixes
*(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)`
format, will close that issue when PR gets merged)*
- fixes #
#### Special notes for your reviewer:
**Note:** Not tested against the real GOV (FedRAMP) endpoint — I don't
have access to a FedRAMP account/license key. GOV endpoint resolution is
verified via unit tests + rendered output; end-to-end ingest was
validated only on the US path.
#### Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove
unrelated fields.]
- [x] Chart Version bumped
- [x] Variables are documented in the README.md
- [x] Title of the PR starts with chart name (e.g. `[mychartname]`)
# Release Notes to Publish (nr-k8s-otel-collector)
If this PR contains changes in `nr-k8s-otel-collector`, please complete
the following section. All other charts should ignore this section.
<!--BEGIN-RELEASE-NOTES-->
## 🚀 What's Changed
* Tell the world about the latest changes in the chart.
<!--END-RELEASE-NOTES-->1 parent e040e13 commit 9255d17
3 files changed
Lines changed: 40 additions & 5 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1 | 1 | | |
2 | 2 | | |
3 | 3 | | |
4 | | - | |
5 | | - | |
6 | | - | |
| 4 | + | |
| 5 | + | |
| 6 | + | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1 | 1 | | |
2 | 2 | | |
3 | 3 | | |
4 | | - | |
| 4 | + | |
5 | 5 | | |
6 | 6 | | |
7 | 7 | | |
| |||
14 | 14 | | |
15 | 15 | | |
16 | 16 | | |
17 | | - | |
| 17 | + | |
18 | 18 | | |
Lines changed: 35 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
106 | 106 | | |
107 | 107 | | |
108 | 108 | | |
| 109 | + | |
| 110 | + | |
| 111 | + | |
| 112 | + | |
| 113 | + | |
| 114 | + | |
| 115 | + | |
| 116 | + | |
| 117 | + | |
| 118 | + | |
| 119 | + | |
| 120 | + | |
| 121 | + | |
| 122 | + | |
| 123 | + | |
| 124 | + | |
| 125 | + | |
| 126 | + | |
| 127 | + | |
| 128 | + | |
| 129 | + | |
| 130 | + | |
| 131 | + | |
| 132 | + | |
| 133 | + | |
| 134 | + | |
| 135 | + | |
| 136 | + | |
| 137 | + | |
| 138 | + | |
| 139 | + | |
| 140 | + | |
| 141 | + | |
| 142 | + | |
| 143 | + | |
109 | 144 | | |
110 | 145 | | |
111 | 146 | | |
| |||
0 commit comments