Skip to content

Conversation

@dpacheconr
Copy link
Contributor

Problem Statement

The nr-ebpf-agent chart is configured in nri-bundle with a mismatch between its condition name and chart name:

  • Condition name: newrelic-eapm-agent.enabled
  • Chart name: nr-ebpf-agent

This forces users to maintain two separate configuration sections in their values file:

newrelic-eapm-agent:
  enabled: true

nr-ebpf-agent:
  ebpfAgent:
    image: {...}
  otelCollector:
    image: {...}

This is inconsistent with all other nri-bundle charts, which use a single configuration section.

Solution

Add alias: newrelic-eapm-agent to the nr-ebpf-agent dependency definition in Chart.yaml.

This allows Helm to route both the condition name and chart name to the same chart, enabling users to consolidate configuration under a single section:

newrelic-eapm-agent:
  enabled: true
  ebpfAgent:
    image: {...}
  otelCollector:
    image: {...}

Implementation

This approach:

  • ✅ Mirrors the existing pattern used for pixie-operator-chart alias
  • ✅ Maintains full backward compatibility
  • ✅ Improves configuration UX consistency across nri-bundle
  • ✅ Requires no changes to the nr-ebpf-agent chart itself

Testing

Verified with helm template that configuration under newrelic-eapm-agent: is properly routed to the nr-ebpf-agent chart subchart.

The nr-ebpf-agent chart uses 'newrelic-eapm-agent' as its condition name but
'nr-ebpf-agent' as the chart name, requiring users to maintain two separate
configuration sections in their values. This is inconsistent with how other
nri-bundle subcharts work.

Adding an alias routes both names to the same chart, allowing users to configure
nr-ebpf-agent under a single 'newrelic-eapm-agent:' section, matching the
intuitive pattern used by other charts and improving the configuration experience.

This approach mirrors the existing pattern used for pixie-operator-chart alias.
@dpacheconr dpacheconr requested a review from a team as a code owner November 18, 2025 13:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant