[core] Rename OVERRIDE_NODE_ID_FOR_TESTING to RAYLET_NODE_ID to make it a feature (#28275)

This PR changed the OVERRIDE_NODE_ID_FOR_TESTING to RAYLET_NODE_ID so that this is a feature which can be used to start raylet with a given raylet id by setting os env RAY_RAYLET_NODE_ID.
This commit is contained in:
Yi Cheng 2022-09-05 21:22:06 +00:00 committed by GitHub
parent 63ab063997
commit 10e9422f8f
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
4 changed files with 7 additions and 10 deletions

View file

@ -322,7 +322,7 @@ class FakeMultiNodeProvider(NodeProvider):
ray._private.services.get_node_ip_address()
),
env_vars={
"RAY_OVERRIDE_NODE_ID_FOR_TESTING": next_id,
"RAY_RAYLET_NODE_ID": next_id,
"RAY_OVERRIDE_RESOURCES": json.dumps(resources),
},
)
@ -472,7 +472,7 @@ class FakeMultiNodeDockerProvider(FakeMultiNodeProvider):
host_client_port=self._host_client_port,
resources=resources,
env_vars={
"RAY_OVERRIDE_NODE_ID_FOR_TESTING": node_id,
"RAY_RAYLET_NODE_ID": node_id,
"RAY_OVERRIDE_RESOURCES": resource_str,
**self.provider_config.get("env_vars", {}),
},

View file

@ -675,9 +675,7 @@ def start(
ray_params.gcs_server_port = port
if os.environ.get("RAY_FAKE_CLUSTER"):
ray_params.env_vars = {
"RAY_OVERRIDE_NODE_ID_FOR_TESTING": FAKE_HEAD_NODE_ID
}
ray_params.env_vars = {"RAY_RAYLET_NODE_ID": FAKE_HEAD_NODE_ID}
num_redis_shards = None
# Start Ray on the head node.

View file

@ -29,4 +29,4 @@ RAY_INTERNAL_FLAG(std::string, JOB_ID, "")
RAY_INTERNAL_FLAG(std::string, RAYLET_PID, "")
/// Override the random node ID for testing.
RAY_INTERNAL_FLAG(std::string, OVERRIDE_NODE_ID_FOR_TESTING, "")
RAY_INTERNAL_FLAG(std::string, RAYLET_NODE_ID, "")

View file

@ -64,10 +64,9 @@ Raylet::Raylet(instrumented_io_context &main_service,
std::shared_ptr<gcs::GcsClient> gcs_client,
int metrics_export_port)
: main_service_(main_service),
self_node_id_(
!RayConfig::instance().OVERRIDE_NODE_ID_FOR_TESTING().empty()
? NodeID::FromHex(RayConfig::instance().OVERRIDE_NODE_ID_FOR_TESTING())
: NodeID::FromRandom()),
self_node_id_(!RayConfig::instance().RAYLET_NODE_ID().empty()
? NodeID::FromHex(RayConfig::instance().RAYLET_NODE_ID())
: NodeID::FromRandom()),
gcs_client_(gcs_client),
node_manager_(main_service,
self_node_id_,