[core] upgrade grpc to v1.58.0

bazel/ray_deps_setup.bzl

@@ -277,12 +277,19 @@ def ray_deps_setup():
     auto_http_archive(
         name = "com_github_grpc_grpc",
         # NOTE: If you update this, also update @boringssl's hash.
-        url = "https://github.com/grpc/grpc/archive/refs/tags/v1.57.1.tar.gz",
-        sha256 = "0762f809b9de845e6a7c809cabccad6aa4143479fd43b396611fe5a086c0aeeb",
+        url = "https://github.com/grpc/grpc/archive/refs/tags/v1.58.0.tar.gz",
+        sha256 = "ec64fdab22726d50fc056474dd29401d914cc616f53ab8f2fe4866772881d581",
         patches = [
             "@io_ray//thirdparty/patches:grpc-cython-copts.patch",
+            # Work around bazelbuild/bazel#21592: with layering_check and
+            # non-sandbox/local spawn, clang can record transitive *.cppmap files
+            # in .d files, which Bazel then reports as undeclared direct deps.
+            # Fixed in Bazel 7.3.0. LLVM used the same workaround by disabling
+            # layering_check in Bazel overlays (llvm/llvm-project@5bba176).
+            "@io_ray//thirdparty/patches:grpc-disable-layering-check.patch",
             "@io_ray//thirdparty/patches:grpc-zlib-fdopen.patch",
             "@io_ray//thirdparty/patches:grpc-configurable-thread-count.patch",
+            "@io_ray//thirdparty/patches:grpc-nextresult-cancelled-init.patch",
         ],
     )
 
@@ -324,8 +331,8 @@ def ray_deps_setup():
         # https://github.com/grpc/grpc/blob/1ff1feaa83e071d87c07827b0a317ffac673794f/bazel/grpc_deps.bzl#L189
         # Ensure this rule matches the rule used by grpc's bazel/grpc_deps.bzl
         name = "boringssl",
-        sha256 = "0675a4f86ce5e959703425d6f9063eaadf6b61b7f3399e77a154c0e85bad46b1",
-        url = "https://github.com/google/boringssl/archive/342e805bc1f5dfdd650e3f031686d6c939b095d9.tar.gz",
+        sha256 = "b21994a857a7aa6d5256ffe355c735ad4c286de44c6c81dfc04edc41a8feaeef",
+        url = "https://github.com/google/boringssl/archive/2ff4b968a7e0cfee66d9f151cb95635b43dc1d5b.tar.gz",
     )
 
     # The protobuf version we use to auto generate python and java code.

src/ray/gcs/tests/gcs_health_check_manager_test.cc

@@ -139,10 +139,10 @@ class GcsHealthCheckManagerTest : public ::testing::Test {
   }
 
   instrumented_io_context io_service;
-  std::shared_ptr<gcs::GcsHealthCheckManager> health_check;
   std::unordered_map<NodeID, std::shared_ptr<rpc::GrpcServer>> servers;
   std::unordered_set<NodeID> dead_nodes;
   ray::observability::FakeHistogram fake_health_check_rpc_latency_ms_histogram_;
+  std::shared_ptr<gcs::GcsHealthCheckManager> health_check;
   const int64_t initial_delay_ms = 100;
   const int64_t timeout_ms = 10;
   const int64_t period_ms = 10;

src/ray/raylet/tests/BUILD.bazel

@@ -142,6 +142,10 @@ ray_cc_test(
     name = "node_manager_test",
     size = "small",
     srcs = ["node_manager_test.cc"],
+    features = select({
+        "@platforms//os:windows": ["compiler_param_file"],
+        "//conditions:default": [],
+    }),
     tags = ["team:core"],
     deps = [
         ":util",

thirdparty/patches/grpc-disable-layering-check.patch

@@ -0,0 +1,69 @@
+diff -u BUILD
+--- BUILD
++++ BUILD
+@@ -30,7 +30,3 @@
+ package(
+-    default_visibility = ["//visibility:public"],
+-    features = [
+-        "layering_check",
+-        "-parse_headers",
+-    ],
++    default_visibility = ["//visibility:public"],
+ )
+diff -u src/core/BUILD
+--- src/core/BUILD
++++ src/core/BUILD
+@@ -25,6 +25,3 @@
+ package(
+     default_visibility = ["//:__subpackages__"],
+-    features = [
+-        "layering_check",
+-    ],
+ )
+diff -u src/cpp/ext/gsm/BUILD
+--- src/cpp/ext/gsm/BUILD
++++ src/cpp/ext/gsm/BUILD
+@@ -24,8 +24,5 @@
+ package(
+     default_visibility = ["//visibility:public"],
+-    features = [
+-        "layering_check",
+-    ],
+ )
+
+ grpc_cc_library(
+diff -u src/cpp/ext/gcp/BUILD
+--- src/cpp/ext/gcp/BUILD
++++ src/cpp/ext/gcp/BUILD
+@@ -24,8 +24,5 @@
+ package(
+     default_visibility = ["//visibility:public"],
+-    features = [
+-        "layering_check",
+-    ],
+ )
+
+ grpc_cc_library(
+diff -u src/cpp/ext/otel/BUILD
+--- src/cpp/ext/otel/BUILD
++++ src/cpp/ext/otel/BUILD
+@@ -24,8 +24,5 @@
+ package(
+     default_visibility = ["//visibility:public"],
+-    features = [
+-        "layering_check",
+-    ],
+ )
+
+ grpc_cc_library(
+diff -u test/core/transport/binder/end2end/fuzzers/BUILD
+--- test/core/transport/binder/end2end/fuzzers/BUILD
++++ test/core/transport/binder/end2end/fuzzers/BUILD
+@@ -17,7 +17,4 @@
+
+ grpc_package(
+     name = "test/core/transport/binder/end2end/fuzzers",
+-    features = [
+-        "layering_check",
+-    ],
+ )

thirdparty/patches/grpc-nextresult-cancelled-init.patch

@@ -0,0 +1,15 @@
+diff --git src/core/lib/promise/pipe.h b/src/core/lib/promise/pipe.h
+--- src/core/lib/promise/pipe.h
++++ src/core/lib/promise/pipe.h
+@@ -63,9 +63,9 @@
+ template <typename T>
+ class NextResult final {
+  public:
+-  NextResult() : center_(nullptr) {}
++  NextResult() : center_(nullptr), cancelled_(true) {}
+   explicit NextResult(RefCountedPtr<pipe_detail::Center<T>> center)
+-      : center_(std::move(center)) {
++      : center_(std::move(center)), cancelled_(false) {
+     GPR_ASSERT(center_ != nullptr);
+   }
+   explicit NextResult(bool cancelled)