Skip to content

Enable more configurable values#330

Merged
Tzvonimir merged 1 commit into
mainfrom
tzvonimir/improve-helm-template-values
Mar 20, 2026
Merged

Enable more configurable values#330
Tzvonimir merged 1 commit into
mainfrom
tzvonimir/improve-helm-template-values

Conversation

@Tzvonimir
Copy link
Copy Markdown
Contributor

@Tzvonimir Tzvonimir commented Mar 20, 2026

Summary

  • Extract hardcoded values (container images, resource limits, ports, replicas, probe configs, Prometheus settings) from zxporter and zxporter-netmon Helm chart templates into values.yaml
  • Makes deployments configurable without modifying templates — operators can now override images, resource requests/limits, ports, retention, scrape intervals, and replica counts via standard Helm value overrides

Changes

zxporter chart:

  • Added new values.yaml sections: resources, nodeExporter, kubeStateMetrics, prometheus
  • Templated node-exporter-daemonset.yaml: image, pull policy, port, resources, hostNetwork, hostPID
  • Templated prometheus-deployments.yaml: images, pull policies, replicas, resources, retention for kube-state-metrics, config-reloader, and prometheus server
  • Templated prometheus-configmap.yaml: evaluation_interval, scrape_interval, scrape_timeout
  • Templated prometheus-services.yaml: node-exporter service port
  • Templated deployment.yaml: controller resources

zxporter-netmon chart:

  • Added containerPort to values.yaml under config
  • Templated containerPort in daemonset.yaml

Test plan

  • helm template renders identical output to pre-change templates (verified — only cosmetic YAML serialization differences: key ordering, quote removal)
  • helm lint passes on both charts with no errors
  • Verify overriding individual values (e.g. --set prometheus.replicas=2, --set nodeExporter.port=9102) renders correctly
  • Deploy to a test cluster and confirm all components start successfully

@Tzvonimir Tzvonimir enabled auto-merge (squash) March 20, 2026 10:52
@Tzvonimir Tzvonimir merged commit aa599ee into main Mar 20, 2026
26 checks passed
@Tzvonimir Tzvonimir deleted the tzvonimir/improve-helm-template-values branch March 20, 2026 10:58
Parthiba-Hazra pushed a commit that referenced this pull request May 5, 2026
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.

2 participants