(7/n) delete hyperactor_state (#520)

Summary:
Pull Request resolved: #520

 1. consolidate logging utils into logging.rs
 2. delete the state actor as there is nothing there now. We can bring it back
easily once we start to build it seriously.

Reviewed By: shayne-fletcher

Differential Revision: D78201006

fbshipit-source-id: 2c79d3128e6d099851b8a055c167b3443b25e570

Author: James Sun

hyperactor_mesh/Cargo.toml

@@ -33,9 +33,9 @@ dashmap = { version = "5.5.3", features = ["rayon", "serde"] }
 enum-as-inner = "0.6.0"
 erased-serde = "0.3.27"
 futures = { version = "0.3.30", features = ["async-await", "compat"] }
+hostname = "0.3"
 hyperactor = { version = "0.0.0", path = "../hyperactor" }
 hyperactor_mesh_macros = { version = "0.0.0", path = "../hyperactor_mesh_macros" }
-hyperactor_state = { version = "0.0.0", path = "../hyperactor_state" }
 hyperactor_telemetry = { version = "0.0.0", path = "../hyperactor_telemetry" }
 libc = "0.2.139"
 mockall = "0.13.1"

hyperactor_mesh/src/alloc/process.rs

@@ -50,6 +50,7 @@ use crate::bootstrap;
 use crate::bootstrap::Allocator2Process;
 use crate::bootstrap::Process2Allocator;
 use crate::bootstrap::Process2AllocatorMessage;
+use crate::logging::create_log_writers;
 use crate::shortuuid::ShortUuid;
 
 /// The maximum number of log lines to tail keep for managed processes.
@@ -155,10 +156,7 @@ impl Child {
             Box::new(io::stderr());
 
         // Use the helper function to create both writers at once
-        match hyperactor_state::log_writer::create_log_writers(
-            log_channel,
-            process.id().unwrap_or(0),
-        ) {
+        match create_log_writers(log_channel, process.id().unwrap_or(0)) {
             Ok((stdout_writer, stderr_writer)) => {
                 stdout_tee = stdout_writer;
                 stderr_tee = stderr_writer;