Skip to content

Roll tiled pods on ConfigMap change - #18

Merged
ZohebShaikh merged 0 commit into
envoy-proxyfrom
auto-roll-deployments
Oct 14, 2026
Merged

ZohebShaikh merged 0 commit into
envoy-proxyfrom
auto-roll-deployments

Conversation

@ZohebShaikh

@ZohebShaikh ZohebShaikh commented Oct 14, 2026 •

Copy link
Copy Markdown
Contributor

Adds rollOnConfigChange (default true). When it's on, the tiled pod template gets a checksum/config annotation holding a hash of the ConfigMap, so helm upgrade restarts the pods whenever the config changes. See Automatically Roll Deployments.

Set it to false to leave the pods running when the config changes.

#11 (envoy) is stacked on top of this.

🤖 Generated with Claude Code

@codecov

codecov Bot commented Oct 14, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 61.64%. Comparing base (66d2cda) to head (2d3f173).

Additional details and impacted files
@@             Coverage Diff              @@
##           envoy-proxy      #18   +/-   ##
============================================
  Coverage        61.64%   61.64%           
============================================
  Files                3        3           
  Lines              219      219           
============================================
  Hits               135      135           
  Misses              84       84           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@ZohebShaikh
ZohebShaikh force-pushed the auto-roll-deployments branch from 2d3f173 to 168221a Compare October 14, 2026 12:15
@ZohebShaikh
ZohebShaikh merged commit 168221a into envoy-proxy Oct 14, 2026
@ZohebShaikh ZohebShaikh changed the title Roll deployments on ConfigMap change Roll tiled pods on ConfigMap change Oct 14, 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.

1 participant