feat: Ignore ready and stats listener metrics in shutdown manager calculation by zirain · Pull Request #7985 · envoyproxy/gateway

zirain · 2026-01-19T04:37:36Z

use listener.(.*).downstream_cx_active stats instead of server.total_connections which contains ready and stats listener.

netlify · 2026-01-19T04:37:43Z

✅ Deploy Preview for cerulean-figolla-1f9435 ready!

Name	Link
🔨 Latest commit	`2d9ad56`
🔍 Latest deploy log	https://app.netlify.com/projects/cerulean-figolla-1f9435/deploys/697a320ef3b895000889422c
😎 Deploy Preview	https://deploy-preview-7985--cerulean-figolla-1f9435.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

codecov · 2026-01-19T04:46:59Z

Codecov Report

❌ Patch coverage is 58.13953% with 18 lines in your changes missing coverage. Please review.
✅ Project coverage is 73.90%. Comparing base (15b62c0) to head (2d9ad56).
⚠️ Report is 26 commits behind head on main.

Files with missing lines	Patch %	Lines
internal/cmd/envoy/shutdown_manager.go	58.13%	15 Missing and 3 partials ⚠️

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #7985   +/-   ##
=======================================
  Coverage   73.89%   73.90%           
=======================================
  Files         237      237           
  Lines       35831    35847   +16     
=======================================
+ Hits        26478    26491   +13     
  Misses       7496     7496           
- Partials     1857     1860    +3

☔ View full report in Codecov by Sentry.
📢 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.
📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

internal/cmd/envoy/shutdown_manager.go

arkodg · 2026-01-20T00:48:48Z

internal/cmd/envoy/shutdown_manager.go

-	} else {
-		defer resp.Body.Close()
+	}
+	defer func() {


can there be a case where resp is nil

AFAIK, resp shouldn't be nil when there's no error.

arkodg

thanks @zirain, can we add an e2e or confirm with a manual test that this works, i.e. envoy exits faster than default drain duration

zirain · 2026-01-28T04:33:17Z

thanks @zirain, can we add an e2e or confirm with a manual test that this works, i.e. envoy exits faster than default drain duration

2026-01-28T04:31:27.402Z        INFO    shutdown-manager        envoy/shutdown_manager.go:93    received shutdown ready request
2026-01-28T04:31:27.500Z        INFO    shutdown-manager        envoy/shutdown_manager.go:130   initiating drain with 10 second minimum drain period and 60 second timeout
2026-01-28T04:31:27.503Z        INFO    shutdown-manager        envoy/shutdown_manager.go:244   total downstream connections: 0
2026-01-28T04:31:28.504Z        INFO    shutdown-manager        envoy/shutdown_manager.go:244   total downstream connections: 0
2026-01-28T04:31:29.511Z        INFO    shutdown-manager        envoy/shutdown_manager.go:244   total downstream connections: 0
2026-01-28T04:31:30.515Z        INFO    shutdown-manager        envoy/shutdown_manager.go:244   total downstream connections: 0
2026-01-28T04:31:31.524Z        INFO    shutdown-manager        envoy/shutdown_manager.go:244   total downstream connections: 0
2026-01-28T04:31:32.526Z        INFO    shutdown-manager        envoy/shutdown_manager.go:244   total downstream connections: 0
2026-01-28T04:31:33.529Z        INFO    shutdown-manager        envoy/shutdown_manager.go:244   total downstream connections: 0
2026-01-28T04:31:34.530Z        INFO    shutdown-manager        envoy/shutdown_manager.go:244   total downstream connections: 0
2026-01-28T04:31:35.533Z        INFO    shutdown-manager        envoy/shutdown_manager.go:244   total downstream connections: 0
2026-01-28T04:31:36.538Z        INFO    shutdown-manager        envoy/shutdown_manager.go:244   total downstream connections: 0
2026-01-28T04:31:37.541Z        INFO    shutdown-manager        envoy/shutdown_manager.go:244   total downstream connections: 0
2026-01-28T04:31:37.541Z        INFO    shutdown-manager        envoy/shutdown_manager.go:149   minimum drain period reached; will exit when total connections reaches 0
2026-01-28T04:31:37.541Z        INFO    shutdown-manager        envoy/shutdown_manager.go:157   drain sequence completed
2026-01-28T04:31:37.553Z        INFO    shutdown-manager        envoy/shutdown_manager.go:67    received terminated
2026-01-28T04:31:38.427Z        INFO    shutdown-manager        envoy/shutdown_manager.go:112   shutdown readiness detected

zhaohuabing

Non-blocking nit: it could help later readers if we can explain each pattern in the skipConnectionRE.

zirain · 2026-01-28T07:14:03Z

wait #8090

…culation Signed-off-by: zirain <[email protected]>

Signed-off-by: zirain <[email protected]>

cnvergence · 2026-01-28T17:37:19Z

/retest

…culation (envoyproxy#7985) * feat: Ignore ready and stats listener metrics in shutdown manager calculation Signed-off-by: zirain <[email protected]> * fix Signed-off-by: zirain <[email protected]> * fix Signed-off-by: zirain <[email protected]> * refactor Signed-off-by: zirain <[email protected]> * remove USE_SERVER_CONNECTIONS Signed-off-by: zirain <[email protected]> * address review comment Signed-off-by: zirain <[email protected]> * display the real value Signed-off-by: zirain <[email protected]> * comment for worker thread Signed-off-by: zirain <[email protected]> --------- Signed-off-by: zirain <[email protected]> Signed-off-by: Sadmi Bouhafs <[email protected]>

…culation (envoyproxy#7985) * feat: Ignore ready and stats listener metrics in shutdown manager calculation Signed-off-by: zirain <[email protected]> * fix Signed-off-by: zirain <[email protected]> * fix Signed-off-by: zirain <[email protected]> * refactor Signed-off-by: zirain <[email protected]> * remove USE_SERVER_CONNECTIONS Signed-off-by: zirain <[email protected]> * address review comment Signed-off-by: zirain <[email protected]> * display the real value Signed-off-by: zirain <[email protected]> * comment for worker thread Signed-off-by: zirain <[email protected]> --------- Signed-off-by: zirain <[email protected]>

* feat: Ignore ready and stats listener metrics in shutdown manager calculation (#7985) * feat: Ignore ready and stats listener metrics in shutdown manager calculation Signed-off-by: zirain <[email protected]> * fix Signed-off-by: zirain <[email protected]> * fix Signed-off-by: zirain <[email protected]> * refactor Signed-off-by: zirain <[email protected]> * remove USE_SERVER_CONNECTIONS Signed-off-by: zirain <[email protected]> * address review comment Signed-off-by: zirain <[email protected]> * display the real value Signed-off-by: zirain <[email protected]> * comment for worker thread Signed-off-by: zirain <[email protected]> --------- Signed-off-by: zirain <[email protected]> * fix: custom response should be put at the first of the filter chain (#8061) * fix: custom response should be put before oauth2 Signed-off-by: Huabing (Robin) Zhao <[email protected]> * move the custom response filter to first Signed-off-by: Huabing (Robin) Zhao <[email protected]> * add release note Signed-off-by: Huabing (Robin) Zhao <[email protected]> --------- Signed-off-by: Huabing (Robin) Zhao <[email protected]> * fix: route idle timeout (#8058) * fix: route idle timeout Signed-off-by: Huabing (Robin) Zhao <[email protected]> * address comments Signed-off-by: Huabing (Robin) Zhao <[email protected]> * add test Signed-off-by: Huabing (Robin) Zhao <[email protected]> --------- Signed-off-by: Huabing (Robin) Zhao <[email protected]> * fix: remove global logger in message package (#8131) * fix: remove global logger in message package Signed-off-by: zirain <[email protected]> * fix gen Signed-off-by: zirain <[email protected]> * fix: controller cache-sync readiness check (#7430) Signed-off-by: zirain <[email protected]> * release notes for v1.5.9 (#8219) * release notes for v1.5.9 Signed-off-by: zirain <[email protected]> * update Signed-off-by: zirain <[email protected]> --------- Signed-off-by: zirain <[email protected]> * update VERSION Signed-off-by: zirain <[email protected]> * update release notes Signed-off-by: zirain <[email protected]> * update Signed-off-by: zirain <[email protected]> --------- Signed-off-by: zirain <[email protected]> Signed-off-by: Huabing (Robin) Zhao <[email protected]> Co-authored-by: Huabing (Robin) Zhao <[email protected]> Co-authored-by: Isaac Wilson <[email protected]>

* fix(status): align BackendTLSPolicy ResolvedRefs reason with Gateway API (#7793) * fix(status): align BackendTLSPolicy ResolvedRefs reason with Gateway API Signed-off-by: Aditya7880900936 <[email protected]> * fix(gatewayapi): use accurate error for missing CA bundle in BackendTLSPolicy Signed-off-by: Aditya7880900936 <[email protected]> * gatewayapi: fix BackendTLSPolicy status reasons for invalid CA refs Signed-off-by: Aditya7880900936 <[email protected]> * Update internal/gatewayapi/backendtlspolicy.go Co-authored-by: Arko Dasgupta <[email protected]> Signed-off-by: Aditya Sanskar Srivastav <[email protected]> * gatewayapi: align BackendTLSPolicy invalid CA status and formatting Signed-off-by: Aditya7880900936 <[email protected]> * gatewayapi: align BackendTLSPolicy invalid CA error message with validation output Signed-off-by: Aditya7880900936 <[email protected]> * testdata: regenerate BackendTLSPolicy invalid CA output Signed-off-by: Aditya7880900936 <[email protected]> * fix(gatewayapi): keep Accepted reason as NoValidCACertificate for invalid CA ref kind Signed-off-by: Aditya7880900936 <[email protected]> * chore(gatewayapi): fix import grouping in BackendTLSPolicy Signed-off-by: Aditya7880900936 <[email protected]> --------- Signed-off-by: Aditya7880900936 <[email protected]> Signed-off-by: Aditya Sanskar Srivastav <[email protected]> Co-authored-by: Arko Dasgupta <[email protected]> * feat: Ignore ready and stats listener metrics in shutdown manager calculation (#7985) * feat: Ignore ready and stats listener metrics in shutdown manager calculation Signed-off-by: zirain <[email protected]> * fix Signed-off-by: zirain <[email protected]> * fix Signed-off-by: zirain <[email protected]> * refactor Signed-off-by: zirain <[email protected]> * remove USE_SERVER_CONNECTIONS Signed-off-by: zirain <[email protected]> * address review comment Signed-off-by: zirain <[email protected]> * display the real value Signed-off-by: zirain <[email protected]> * comment for worker thread Signed-off-by: zirain <[email protected]> --------- Signed-off-by: zirain <[email protected]> * fix: custom response should be put at the first of the filter chain (#8061) * fix: custom response should be put before oauth2 Signed-off-by: Huabing (Robin) Zhao <[email protected]> * move the custom response filter to first Signed-off-by: Huabing (Robin) Zhao <[email protected]> * add release note Signed-off-by: Huabing (Robin) Zhao <[email protected]> --------- Signed-off-by: Huabing (Robin) Zhao <[email protected]> * fix: route idle timeout (#8058) * fix: route idle timeout Signed-off-by: Huabing (Robin) Zhao <[email protected]> * address comments Signed-off-by: Huabing (Robin) Zhao <[email protected]> * add test Signed-off-by: Huabing (Robin) Zhao <[email protected]> --------- Signed-off-by: Huabing (Robin) Zhao <[email protected]> * fix: remove global logger in message package (#8131) * fix: remove global logger in message package Signed-off-by: zirain <[email protected]> * fix: TCPRoute mTLS didn't work (#8152) * fix: remove auto HTTP config on TCP cluster Signed-off-by: zirain <[email protected]> * fix lint Signed-off-by: zirain <[email protected]> * add e2e Signed-off-by: zirain <[email protected]> * fix e2e Signed-off-by: zirain <[email protected]> * fix comment Signed-off-by: zirain <[email protected]> * fix Signed-off-by: zirain <[email protected]> * fix resource name Signed-off-by: zirain <[email protected]> * address Arko's comment Signed-off-by: zirain <[email protected]> --------- Signed-off-by: zirain <[email protected]> * fix: continue processing the remaining xDS with invalid EnvoyPatchPolicies (#8153) continue processing the remaining xDS with invalid EnvoyPatchPolicies Signed-off-by: Huabing (Robin) Zhao <[email protected]> * fix gen Signed-off-by: zirain <[email protected]> * fix gen Signed-off-by: zirain <[email protected]> * fix: controller cache-sync readiness check (#7430) Signed-off-by: zirain <[email protected]> * fix gen Signed-off-by: zirain <[email protected]> * release notes for v1.6.4 (#8221) * release notes for v1.6.4 Signed-off-by: zirain <[email protected]> * update Signed-off-by: zirain <[email protected]> --------- Signed-off-by: zirain <[email protected]> * update VERSION Signed-off-by: zirain <[email protected]> * update release notes Signed-off-by: zirain <[email protected]> * update Signed-off-by: zirain <[email protected]> --------- Signed-off-by: Aditya7880900936 <[email protected]> Signed-off-by: Aditya Sanskar Srivastav <[email protected]> Signed-off-by: zirain <[email protected]> Signed-off-by: Huabing (Robin) Zhao <[email protected]> Co-authored-by: Aditya Sanskar Srivastav <[email protected]> Co-authored-by: Arko Dasgupta <[email protected]> Co-authored-by: Huabing (Robin) Zhao <[email protected]> Co-authored-by: Isaac Wilson <[email protected]>

zirain requested a review from a team as a code owner January 19, 2026 04:37

zirain changed the title ~~feat: Ignore ready and stats listener metrics in shutdown manager cal…~~ feat: Ignore ready and stats listener metrics in shutdown manager calculation Jan 19, 2026

arkodg reviewed Jan 20, 2026

View reviewed changes

internal/cmd/envoy/shutdown_manager.go Outdated Show resolved Hide resolved

zirain force-pushed the fix-7947 branch from 2e002c6 to 724b789 Compare January 20, 2026 00:46

zirain requested a review from arkodg January 20, 2026 00:48

arkodg reviewed Jan 20, 2026

View reviewed changes

zirain added this to the v1.7.0-rc.1 Release milestone Jan 28, 2026

arkodg previously approved these changes Jan 28, 2026

View reviewed changes

arkodg requested review from a team January 28, 2026 03:54

zirain dismissed arkodg’s stale review via 6746eee January 28, 2026 04:32

zirain mentioned this pull request Jan 28, 2026

Add e2e to ensure that shutdown manager worked as expected #8087

Open

arko-oai approved these changes Jan 28, 2026

View reviewed changes

arkodg previously approved these changes Jan 28, 2026

View reviewed changes

zhaohuabing previously approved these changes Jan 28, 2026

View reviewed changes

zirain dismissed stale reviews from zhaohuabing and arkodg via 48d51c8 January 28, 2026 05:09

zhaohuabing approved these changes Jan 28, 2026

View reviewed changes

zirain force-pushed the fix-7947 branch from 48d51c8 to bcc1c24 Compare January 28, 2026 05:14

zhaohuabing approved these changes Jan 28, 2026

View reviewed changes

cnvergence approved these changes Jan 28, 2026

View reviewed changes

zirain added 5 commits January 28, 2026 23:58

feat: Ignore ready and stats listener metrics in shutdown manager cal…

5b470db

…culation Signed-off-by: zirain <[email protected]>

fix

2dc8876

Signed-off-by: zirain <[email protected]>

fix

fd38e68

Signed-off-by: zirain <[email protected]>

refactor

70c05a0

Signed-off-by: zirain <[email protected]>

remove USE_SERVER_CONNECTIONS

df8600f

Signed-off-by: zirain <[email protected]>

zirain added 3 commits January 28, 2026 23:58

address review comment

b055d92

Signed-off-by: zirain <[email protected]>

display the real value

0719791

Signed-off-by: zirain <[email protected]>

comment for worker thread

2d9ad56

Signed-off-by: zirain <[email protected]>

zirain force-pushed the fix-7947 branch from bcc1c24 to 2d9ad56 Compare January 28, 2026 15:58

jukie approved these changes Jan 28, 2026

View reviewed changes

zirain merged commit 762ea33 into envoyproxy:main Jan 28, 2026
57 of 59 checks passed

zirain deleted the fix-7947 branch January 28, 2026 22:21

arkodg added cherrypick/release-v1.6.4 cherrypick/release-v1.5.9 labels Jan 31, 2026

zirain mentioned this pull request Feb 9, 2026

[release-1.5] cherry pick for v1.5.9 #8217

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

feat: Ignore ready and stats listener metrics in shutdown manager calculation#7985

feat: Ignore ready and stats listener metrics in shutdown manager calculation#7985
zirain merged 8 commits intoenvoyproxy:mainfrom
zirain:fix-7947

zirain commented Jan 19, 2026 •

edited

Loading

Uh oh!

netlify bot commented Jan 19, 2026 •

edited

Loading

Uh oh!

codecov bot commented Jan 19, 2026 •

edited

Loading

Uh oh!

Uh oh!

arkodg Jan 20, 2026

Uh oh!

zirain Jan 20, 2026

Uh oh!

arkodg left a comment

Uh oh!

zirain commented Jan 28, 2026

Uh oh!

zhaohuabing left a comment •

edited

Loading

Uh oh!

zirain commented Jan 28, 2026

Uh oh!

cnvergence commented Jan 28, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

6 participants

Conversation

zirain commented Jan 19, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

netlify bot commented Jan 19, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

✅ Deploy Preview for cerulean-figolla-1f9435 ready!

Uh oh!

codecov bot commented Jan 19, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Codecov Report

Uh oh!

Uh oh!

arkodg Jan 20, 2026

Choose a reason for hiding this comment

Uh oh!

zirain Jan 20, 2026

Choose a reason for hiding this comment

Uh oh!

arkodg left a comment

Choose a reason for hiding this comment

Uh oh!

zirain commented Jan 28, 2026

Uh oh!

zhaohuabing left a comment • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

zirain commented Jan 28, 2026

Uh oh!

cnvergence commented Jan 28, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

6 participants

zirain commented Jan 19, 2026 •

edited

Loading

netlify bot commented Jan 19, 2026 •

edited

Loading

codecov bot commented Jan 19, 2026 •

edited

Loading

zhaohuabing left a comment •

edited

Loading