Revert "Enable GRPC_ERROR_IS_ABSEIL_STATUS (#28898)" (#28939)

This reverts commit f00e842609.

BUILD

@@ -143,6 +143,11 @@ config_setting(
     values = {"define": "use_strict_warning=true"},
 )
 
+config_setting(
+    name = "use_abseil_status",
+    values = {"define": "use_abseil_status=true"},
+)
+
 python_config_settings()
 
 # This should be updated along with build_handwritten.yaml

bazel/grpc_build_system.bzl

@@ -177,6 +177,10 @@ def grpc_cc_library(
                       "//:grpc_allow_exceptions": ["GRPC_ALLOW_EXCEPTIONS=1"],
                       "//:grpc_disallow_exceptions": ["GRPC_ALLOW_EXCEPTIONS=0"],
                       "//conditions:default": [],
+                  }) +
+                  select({
+                      "//:use_abseil_status": ["GRPC_ERROR_IS_ABSEIL_STATUS=1"],
+                      "//conditions:default": [],
                   }),
         hdrs = hdrs + public_hdrs,
         deps = deps + _get_external_deps(external_deps),

include/grpc/impl/codegen/port_platform.h

@@ -51,7 +51,9 @@
 /*
  * Defines GRPC_ERROR_IS_ABSEIL_STATUS to use absl::Status for grpc_error_handle
  */
-#define GRPC_ERROR_IS_ABSEIL_STATUS 1
+#ifndef GRPC_ERROR_IS_ABSEIL_STATUS
+// #define GRPC_ERROR_IS_ABSEIL_STATUS 1
+#endif
 
 /* Get windows.h included everywhere (we need it) */
 #if defined(_WIN64) || defined(WIN64) || defined(_WIN32) || defined(WIN32)

src/python/grpcio_tests/tests/unit/_rpc_part_1_test.py

@@ -114,8 +114,10 @@ class RPCPart1Test(BaseRPCTest, unittest.TestCase):
         # sanity checks on to make sure returned string contains default members
         # of the error
         debug_error_string = exception_context.exception.debug_error_string()
-        self.assertIn('grpc_status', debug_error_string)
+        self.assertIn('created', debug_error_string)
-        self.assertIn('grpc_message', debug_error_string)
+        self.assertIn('description', debug_error_string)
+        self.assertIn('file', debug_error_string)
+        self.assertIn('file_line', debug_error_string)
 
     def testFailedUnaryRequestFutureUnaryResponse(self):
         request = b'\x37\x17'

src/ruby/lib/grpc/errors.rb

@@ -43,7 +43,7 @@ module GRPC
                    debug_error_string = nil)
       exception_message = "#{code}:#{details}"
       if debug_error_string
-        exception_message += ". debug_error_string:{#{debug_error_string}}"
+        exception_message += ". debug_error_string:#{debug_error_string}"
       end
       super(exception_message)
       @code = code

tools/internal_ci/linux/grpc_bazel_build_in_docker.sh

@@ -38,6 +38,13 @@ bazel build --define=use_strict_warning=true \
 	//examples/... \
 	-//examples/android/binder/...
 
+# TODO(veblush): Remove this test after migration to abseil-status is done.
+bazel build --define=use_strict_warning=true --define=use_abseil_status=true \
+	-- \
+	//src/core/... \
+	//src/compiler/... \
+	//test/...
+
 # TODO(jtattersmusch): Adding a build here for --define=grpc_no_xds is not ideal
 # and we should find a better place for this. Refer
 # https://github.com/grpc/grpc/pull/24536#pullrequestreview-517466531 for more