(trafficserver) branch master updated: Increment parent mark down if config allows (#12318)

[lserris]

This is an automated email from the ASF dual-hosted git repository.

lserris pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/trafficserver.git


The following commit(s) were added to refs/heads/master by this push:
     new e0909abeeb Increment parent mark down if config allows (#12318)
e0909abeeb is described below

commit e0909abeeb04654e04ed9493f5c0a717d29c3bfa
Author: Serris Lew <serrisn...@gmail.com>
AuthorDate: Tue Jul 1 09:00:04 2025 -0700

    Increment parent mark down if config allows (#12318)
    
    Co-authored-by: Serris Lew <lser...@apple.com>
---
 src/proxy/http/HttpTransact.cc | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/src/proxy/http/HttpTransact.cc b/src/proxy/http/HttpTransact.cc
index 778c97d4fa..3ff94ca5d4 100644
--- a/src/proxy/http/HttpTransact.cc
+++ b/src/proxy/http/HttpTransact.cc
@@ -237,7 +237,6 @@ findParent(HttpTransact::State *s)
 inline static void
 markParentDown(HttpTransact::State *s)
 {
-  Metrics::Counter::increment(http_rsb.total_parent_marked_down_count);
   url_mapping *mp = s->url_map.getMapping();
 
   TxnDbg(dbg_ctl_http_trans, "enable_parent_timeout_markdowns: %d, 
disable_parent_markdowns: %d",
@@ -251,6 +250,8 @@ markParentDown(HttpTransact::State *s)
   if (s->current.state == HttpTransact::INACTIVE_TIMEOUT && 
s->txn_conf->enable_parent_timeout_markdowns == 0) {
     return;
   }
+  // Increment metric when config allows ATS to mark parent down
+  Metrics::Counter::increment(http_rsb.total_parent_marked_down_count);
 
   if (s->response_action.handled) {
     // Do nothing. If a plugin handled the response, let it handle markdown.