fix : disable afe_connectivity_error_count metric (#2417)

surbhigarg92 · gcf-owl-bot[bot] · web-flow · commit 24ad294f1de4 · 2025-09-12T02:02:41.000+05:30
* fix : disable afe_connectivity_error_count metric * fix: disable afe_connectivity_error_count metric * 🦉 Updates from OwlBot post-processor See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md * 🦉 Updates from OwlBot post-processor See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md --------- Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
diff --git a/handwritten/spanner/src/metrics/interceptor.ts b/handwritten/spanner/src/metrics/interceptor.ts
@@ -67,7 +67,9 @@ export const MetricInterceptor = (options, nextCall) => {
           if (metricsTracer?.afeLatency) {
             metricsTracer?.recordAfeLatency(status.code);
           } else {
-            metricsTracer?.recordAfeConnectivityErrorCount(status.code);
+            // Disable afe_connectivity_error_count metric as AFE header is disabled in backend
+            // currently.
+            // metricsTracer?.recordAfeConnectivityErrorCount(status.code);
           }
         },
       };
diff --git a/handwritten/spanner/test/metrics/interceptor.ts b/handwritten/spanner/test/metrics/interceptor.ts
@@ -193,7 +193,7 @@ describe('MetricInterceptor', () => {
       );
     });
 
-    it('AFE Metrics - Connectivity Error Count', () => {
+    it.skip('AFE Metrics - Connectivity Error Count', () => {
       const interceptingCall = MetricInterceptor(mockOptions, mockNextCall);
       interceptingCall.start(testMetadata, mockListener);
 
diff --git a/handwritten/spanner/test/metrics/metrics.ts b/handwritten/spanner/test/metrics/metrics.ts
@@ -457,10 +457,6 @@ describe('Test metrics with mock server', () => {
         resourceMetrics,
         METRIC_NAME_GFE_CONNECTIVITY_ERROR_COUNT,
       );
-      const afeConnectivityErrorCountData = getMetricData(
-        resourceMetrics,
-        METRIC_NAME_AFE_CONNECTIVITY_ERROR_COUNT,
-      );
 
       // Verify GFE AFE latency doesn't exist
       assert.ok(!hasMetricData(resourceMetrics, METRIC_NAME_GFE_LATENCIES));
@@ -486,10 +482,6 @@ describe('Test metrics with mock server', () => {
           getAggregatedValue(connectivityErrorCountData, attributes),
           1,
         );
-        assert.strictEqual(
-          getAggregatedValue(afeConnectivityErrorCountData, attributes),
-          1,
-        );
       });
     });
 
