Skip to content

feat(chart): add podLabels support#105

Merged
wcatz merged 1 commit intomasterfrom
feat/pod-labels
Mar 13, 2026
Merged

feat(chart): add podLabels support#105
wcatz merged 1 commit intomasterfrom
feat/pod-labels

Conversation

@wcatz
Copy link
Copy Markdown
Owner

@wcatz wcatz commented Mar 13, 2026

Summary

  • Add podLabels support to the deployment template for custom pod label injection
  • Enables helmfile label conformity — app, network, group labels on pods match helmfile release labels
  • Prometheus scrape targets can now correlate directly with helmfile selectors
  • Bump chart version to 0.7.17

Test plan

  • helm template with podLabels set renders labels on pod spec
  • helm template with empty/default podLabels produces no change

Summary by CodeRabbit

  • New Features

    • Added support for custom pod labels configuration in deployment metadata.
  • Chores

    • Bumped chart version to 0.7.17.

@wcatz wcatz merged commit 05505bd into master Mar 13, 2026
3 checks passed
@wcatz wcatz deleted the feat/pod-labels branch March 13, 2026 14:27
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Mar 13, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 2e141cb2-ef05-49e3-bf35-23eaa8859ba1

📥 Commits

Reviewing files that changed from the base of the PR and between 63115c5 and 5ae7cb6.

📒 Files selected for processing (3)
  • helm-chart/Chart.yaml
  • helm-chart/templates/deployment.yaml
  • helm-chart/values.yaml

📝 Walkthrough

Walkthrough

This pull request adds support for custom pod labels in the Helm chart deployment. It introduces a new podLabels configuration value that conditionally renders custom labels into the pod template metadata, increments the chart version, and defines an empty podLabels map in the default values.

Changes

Cohort / File(s) Summary
Version Management
helm-chart/Chart.yaml
Bumps chart version from 0.7.16 to 0.7.17.
Pod Labels Feature
helm-chart/templates/deployment.yaml, helm-chart/values.yaml
Adds conditional rendering of custom podLabels in pod metadata and introduces new empty podLabels configuration key to values, allowing users to specify custom labels for pods.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Possibly related PRs

  • fix(chart version): bump #6: Both PRs bump the chart version in helm-chart/Chart.yaml, indicating a sequence of incremental version releases.

Poem

🐰 A patch to labels small,
Where pods now dress in custom shawl,
With podLabels in place so neat,
Our Helm deployments are complete! ✨

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch feat/pod-labels
📝 Coding Plan
  • Generate coding plan for human review comments

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

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