Revert "[promises] Enable promise-based calls on server side for OSS build" (#33989)

Reverts grpc/grpc#33945

bazel/experiments.bzl

@@ -27,8 +27,12 @@ EXPERIMENTS = {
             "core_end2end_test": [
                 "event_engine_listener",
                 "promise_based_client_call",
+                "promise_based_server_call",
                 "unique_metadata_strings",
             ],
+            "cpp_end2end_test": [
+                "promise_based_server_call",
+            ],
             "endpoint_test": [
                 "tcp_frame_size_tuning",
                 "tcp_rcv_lowat",
@@ -44,26 +48,22 @@ EXPERIMENTS = {
             "lame_client_test": [
                 "promise_based_client_call",
             ],
+            "logging_test": [
+                "promise_based_server_call",
+            ],
             "resource_quota_test": [
                 "free_large_allocator",
                 "memory_pressure_controller",
                 "unconstrained_max_quota_buffer_size",
             ],
+            "xds_end2end_test": [
+                "promise_based_server_call",
+            ],
         },
         "on": {
             "core_end2end_test": [
-                "promise_based_server_call",
                 "work_stealing",
             ],
-            "cpp_end2end_test": [
-                "promise_based_server_call",
-            ],
-            "logging_test": [
-                "promise_based_server_call",
-            ],
-            "xds_end2end_test": [
-                "promise_based_server_call",
-            ],
         },
     },
     "ios": {
@@ -76,8 +76,12 @@ EXPERIMENTS = {
             "core_end2end_test": [
                 "event_engine_listener",
                 "promise_based_client_call",
+                "promise_based_server_call",
                 "unique_metadata_strings",
             ],
+            "cpp_end2end_test": [
+                "promise_based_server_call",
+            ],
             "endpoint_test": [
                 "tcp_frame_size_tuning",
                 "tcp_rcv_lowat",
@@ -93,26 +97,22 @@ EXPERIMENTS = {
             "lame_client_test": [
                 "promise_based_client_call",
             ],
+            "logging_test": [
+                "promise_based_server_call",
+            ],
             "resource_quota_test": [
                 "free_large_allocator",
                 "memory_pressure_controller",
                 "unconstrained_max_quota_buffer_size",
             ],
+            "xds_end2end_test": [
+                "promise_based_server_call",
+            ],
         },
         "on": {
             "core_end2end_test": [
-                "promise_based_server_call",
                 "work_stealing",
             ],
-            "cpp_end2end_test": [
-                "promise_based_server_call",
-            ],
-            "logging_test": [
-                "promise_based_server_call",
-            ],
-            "xds_end2end_test": [
-                "promise_based_server_call",
-            ],
         },
     },
     "posix": {
@@ -129,8 +129,12 @@ EXPERIMENTS = {
                 "event_engine_client",
                 "event_engine_listener",
                 "promise_based_client_call",
+                "promise_based_server_call",
                 "unique_metadata_strings",
             ],
+            "cpp_end2end_test": [
+                "promise_based_server_call",
+            ],
             "endpoint_test": [
                 "tcp_frame_size_tuning",
                 "tcp_rcv_lowat",
@@ -149,6 +153,9 @@ EXPERIMENTS = {
             "lame_client_test": [
                 "promise_based_client_call",
             ],
+            "logging_test": [
+                "promise_based_server_call",
+            ],
             "resolver_component_tests_runner_invoker": [
                 "event_engine_dns",
             ],
@@ -157,21 +164,14 @@ EXPERIMENTS = {
                 "memory_pressure_controller",
                 "unconstrained_max_quota_buffer_size",
             ],
+            "xds_end2end_test": [
+                "promise_based_server_call",
+            ],
         },
         "on": {
             "core_end2end_test": [
-                "promise_based_server_call",
                 "work_stealing",
             ],
-            "cpp_end2end_test": [
-                "promise_based_server_call",
-            ],
-            "logging_test": [
-                "promise_based_server_call",
-            ],
-            "xds_end2end_test": [
-                "promise_based_server_call",
-            ],
         },
     },
 }

src/core/lib/experiments/experiments.cc

@@ -131,7 +131,7 @@ const ExperimentMetadata g_experiment_metadata[] = {
     {"free_large_allocator", description_free_large_allocator,
      additional_constraints_free_large_allocator, false, true},
     {"promise_based_server_call", description_promise_based_server_call,
-     additional_constraints_promise_based_server_call, true, true},
+     additional_constraints_promise_based_server_call, false, true},
     {"transport_supplies_client_latency",
      description_transport_supplies_client_latency,
      additional_constraints_transport_supplies_client_latency, false, true},
@@ -273,7 +273,7 @@ const ExperimentMetadata g_experiment_metadata[] = {
     {"free_large_allocator", description_free_large_allocator,
      additional_constraints_free_large_allocator, false, true},
     {"promise_based_server_call", description_promise_based_server_call,
-     additional_constraints_promise_based_server_call, true, true},
+     additional_constraints_promise_based_server_call, false, true},
     {"transport_supplies_client_latency",
      description_transport_supplies_client_latency,
      additional_constraints_transport_supplies_client_latency, false, true},
@@ -415,7 +415,7 @@ const ExperimentMetadata g_experiment_metadata[] = {
     {"free_large_allocator", description_free_large_allocator,
      additional_constraints_free_large_allocator, false, true},
     {"promise_based_server_call", description_promise_based_server_call,
-     additional_constraints_promise_based_server_call, true, true},
+     additional_constraints_promise_based_server_call, false, true},
     {"transport_supplies_client_latency",
      description_transport_supplies_client_latency,
      additional_constraints_transport_supplies_client_latency, false, true},

src/core/lib/experiments/experiments.h

@@ -69,8 +69,7 @@ inline bool IsEventEngineClientEnabled() { return false; }
 inline bool IsMonitoringExperimentEnabled() { return true; }
 inline bool IsPromiseBasedClientCallEnabled() { return false; }
 inline bool IsFreeLargeAllocatorEnabled() { return false; }
-#define GRPC_EXPERIMENT_IS_INCLUDED_PROMISE_BASED_SERVER_CALL
-inline bool IsPromiseBasedServerCallEnabled() { return true; }
+inline bool IsPromiseBasedServerCallEnabled() { return false; }
 inline bool IsTransportSuppliesClientLatencyEnabled() { return false; }
 inline bool IsEventEngineListenerEnabled() { return false; }
 inline bool IsScheduleCancellationOverWriteEnabled() { return false; }
@@ -104,8 +103,7 @@ inline bool IsEventEngineClientEnabled() { return false; }
 inline bool IsMonitoringExperimentEnabled() { return true; }
 inline bool IsPromiseBasedClientCallEnabled() { return false; }
 inline bool IsFreeLargeAllocatorEnabled() { return false; }
-#define GRPC_EXPERIMENT_IS_INCLUDED_PROMISE_BASED_SERVER_CALL
-inline bool IsPromiseBasedServerCallEnabled() { return true; }
+inline bool IsPromiseBasedServerCallEnabled() { return false; }
 inline bool IsTransportSuppliesClientLatencyEnabled() { return false; }
 inline bool IsEventEngineListenerEnabled() { return false; }
 inline bool IsScheduleCancellationOverWriteEnabled() { return false; }
@@ -139,8 +137,7 @@ inline bool IsEventEngineClientEnabled() { return false; }
 inline bool IsMonitoringExperimentEnabled() { return true; }
 inline bool IsPromiseBasedClientCallEnabled() { return false; }
 inline bool IsFreeLargeAllocatorEnabled() { return false; }
-#define GRPC_EXPERIMENT_IS_INCLUDED_PROMISE_BASED_SERVER_CALL
-inline bool IsPromiseBasedServerCallEnabled() { return true; }
+inline bool IsPromiseBasedServerCallEnabled() { return false; }
 inline bool IsTransportSuppliesClientLatencyEnabled() { return false; }
 inline bool IsEventEngineListenerEnabled() { return false; }
 inline bool IsScheduleCancellationOverWriteEnabled() { return false; }

src/core/lib/experiments/rollouts.yaml

@@ -61,7 +61,7 @@
 - name: free_large_allocator
   default: false
 - name: promise_based_server_call
-  default: true
+  default: false
 - name: transport_supplies_client_latency
   default: false
 - name: event_engine_listener