fix inconsistent logging crate names

Signed-off-by: Simon Davies <[email protected]>

Author: simongdavies

docs/hyperlight-metrics-logs-and-traces.md

@@ -66,13 +66,13 @@ In the [examples/tracing](../src/hyperlight_host/examples/tracing) directory, th
 #### Linux
 
 ```bash
-RUST_LOG='none,hyperlight-host=info,tracing=info' cargo run --example tracing
+RUST_LOG='none,hyperlight_host=info,tracing=info' cargo run --example tracing
 ```
 
 #### Windows
 
 ```powershell
-$env:RUST_LOG='none,hyperlight-host=info,tracing=info'; cargo run --example tracing
+$env:RUST_LOG='none,hyperlight_host=info,tracing=info'; cargo run --example tracing
 ```
 
 ### Using OTLP exporter and Jaeger

src/hyperlight_host/src/sandbox/outb.rs

@@ -88,7 +88,7 @@ pub(super) fn outb_log(mgr: &mut SandboxMemoryManager<HostSharedMemory>) -> Resu
             &Record::builder()
                 .args(format_args!("{}", log_data.message))
                 .level(record_level)
-                .target("hyperlight-guest")
+                .target("hyperlight_guest")
                 .file(source_file)
                 .line(line)
                 .module_path(source)
@@ -100,7 +100,7 @@ pub(super) fn outb_log(mgr: &mut SandboxMemoryManager<HostSharedMemory>) -> Resu
             &Record::builder()
                 .args(format_args!("{}", log_data.message))
                 .level(record_level)
-                .target("hyperlight-guest")
+                .target("hyperlight_guest")
                 .file(Some(&log_data.source_file))
                 .line(Some(log_data.line))
                 .module_path(Some(&log_data.source))
@@ -442,7 +442,7 @@ mod tests {
                         test_value_as_str(metadata_values_map, "level", expected_level);
                         test_value_as_str(event_values_map, "log.file", "test source file");
                         test_value_as_str(event_values_map, "log.module_path", "test source");
-                        test_value_as_str(event_values_map, "log.target", "hyperlight-guest");
+                        test_value_as_str(event_values_map, "log.target", "hyperlight_guest");
                         count_matching_events += 1;
                     }
                     assert!(

src/hyperlight_host/tests/integration_test.rs

@@ -485,8 +485,8 @@ fn log_message() {
     log_test_messages();
     assert_eq!(5 + num_fixed_trace_log, LOGGER.num_log_calls());
     // The number of enabled calls is the number of times that the enabled function is called
-    // with a target of "hyperlight-guest"
-    // This should be the same as the number of log calls as all the log calls for the "hyperlight-guest" target should be filtered in
+    // with a target of "hyperlight_guest"
+    // This should be the same as the number of log calls as all the log calls for the "hyperlight_guest" target should be filtered in
     // the guest
     assert_eq!(LOGGER.num_log_calls(), LOGGER.num_enabled_calls());
 

src/hyperlight_testing/src/logger.rs

@@ -90,7 +90,7 @@ impl Log for Logger {
         }
 
         LOGCALLS.with(|log_calls| {
-            if record.target().contains("hyperlight-guest") {
+            if record.target().contains("hyperlight_guest") {
                 println!("Thread {:?} {:?}", current().id(), record);
                 println!("Thread {:?} {:?}", current().id(), record.metadata());
             }

src/hyperlight_testing/src/simplelogger.rs

@@ -15,7 +15,7 @@ limitations under the License.
 */
 
 // this is a non threadsafe logger for testing purposes, to test the log messages emitted by the guest.
-// it will only log messages from the hyperlight-guest target. It will not log messages from other targets.
+// it will only log messages from the hyperlight_guest target. It will not log messages from other targets.
 // this target is only used when handling an outb log request from the guest, so this logger will only capture those messages.
 
 use std::sync::Once;
@@ -80,13 +80,13 @@ impl Log for SimpleLogger {
     fn enabled(&self, metadata: &Metadata) -> bool {
         // This allows us to count the actual number of messages that have been logged by the guest
         // because the guest derives its log level from the host log level then the number times that enabled is called for
-        // the "hyperlight-guest" target will be the same as the number of messages logged by the guest.
-        // In other words this function should always return true for the "hyperlight-guest" target.
+        // the "hyperlight_guest" target will be the same as the number of messages logged by the guest.
+        // In other words this function should always return true for the "hyperlight_guest" target.
         unsafe {
-            if metadata.target() == "hyperlight-guest" {
+            if metadata.target() == "hyperlight_guest" {
                 NUMBER_OF_ENABLED_CALLS += 1;
             }
-            metadata.target() == "hyperlight-guest" && metadata.level() <= log::max_level()
+            metadata.target() == "hyperlight_guest" && metadata.level() <= log::max_level()
         }
     }
     fn log(&self, record: &Record) {