Fix guest tracing filter

[dblnz]

This closes #990.
Currently, when tracing guests, the max log level parameter provided to the guest is not used to filter out traces.

This PR adds the filtering capability for the guest tracing and also modifies some of the trace levels in the guest.
For that to work, a change in the integration_tests.rs was necessary to update the number of expected logs in log_message test

[ludfjig]

LGTM, but it would be nice if it was possible to remove all uses of tracing::log::LeverFilter and log::Levelfilter for tracing_core::LevelFilter/tracing::LevelFilter instead

[jsturtevant]

@dblnz would you like to get this into the next release (we have it scheduled for next week (Feb 25th)

[dblnz]

@dblnz would you like to get this into the next release (we have it scheduled for next week (Feb 25th)

Yes, I'm rebasing it to include it in the release

[Copilot]

Pull request overview

This PR fixes issue #990 by implementing proper filtering of guest tracing spans and events based on the max_log_level parameter. Previously, even though the log level was correctly received by guests, only logs were filtered while tracing spans and events were not.

Changes:

• Introduced a new GuestLogFilter enum to unify log level conversions between log::LevelFilter, tracing_core::LevelFilter, and u64 values passed across the host-guest boundary
• Implemented filtering in the GuestSubscriber::enabled() method to properly filter tracing spans and events based on the maximum log level
• Updated trace span levels from Trace to Info for infrastructure calls (halt, dispatch_function, call_guest_function, call_host_function) to ensure they are visible at appropriate log levels and updated test expectations accordingly

Reviewed changes

Copilot reviewed 18 out of 22 changed files in this pull request and generated 2 comments.

Show a summary per file

File	Description
src/hyperlight_common/src/log_level.rs	New file introducing GuestLogFilter enum with conversions to/from log::LevelFilter, tracing_core::LevelFilter, and u64
src/hyperlight_common/src/lib.rs	Adds log_level module export
src/hyperlight_common/Cargo.toml	Adds tracing-core dependency
src/hyperlight_guest_tracing/src/subscriber.rs	Adds max_log_level field and implements filtering in enabled() method
src/hyperlight_guest_tracing/src/lib.rs	Updates init_guest_tracing to accept LevelFilter parameter
src/hyperlight_guest_tracing/Cargo.toml	Adds "log" feature to tracing dependency for log/tracing interop
src/hyperlight_guest_bin/src/lib.rs	Updates to use GuestLogFilter for initialization
src/hyperlight_guest_bin/src/guest_logger.rs	Minor parameter rename for clarity
src/hyperlight_guest_bin/src/guest_function/call.rs	Updates trace levels from Trace to Info and removes explicit parent span references
src/hyperlight_host/src/hypervisor/mod.rs	Removes old get_max_log_level function
src/hyperlight_host/src/hypervisor/hyperlight_vm.rs	Adds new get_max_log_level_filter and get_guest_log_filter functions with proper conversions
src/hyperlight_host/src/sandbox/uninitialized.rs	Updates imports to use tracing_core::LevelFilter
src/hyperlight_host/src/sandbox/trace/context.rs	Updates span levels from trace to info
src/hyperlight_host/tests/integration_test.rs	Updates test expectations to account for new info-level trace logs and converts to use tracing_core::LevelFilter
src/hyperlight_guest/src/exit.rs	Updates halt span level from Trace to Info
src/hyperlight_guest/src/guest_handle/io.rs	Removes explicit parent span reference
src/hyperlight_guest/src/guest_handle/host_comm.rs	Adds instrumentation and updates trace levels
src/tests/rust_guests/simpleguest/src/main.rs	CRITICAL BUG: Incorrect conversion logic in LogMessage function
Cargo.lock files	Updates dependency locks for tracing-core additions

[Copilot]

Pull request overview

Copilot reviewed 17 out of 21 changed files in this pull request and generated 3 comments.

[ludfjig]

Awesome!

[ludfjig]

I know this is not related to this PR, but this code could be simplified. AI came up with this but I'm sure it can be further tweaked (not blocking, feel free to ignore if you wish)

fn get_max_log_level_filter() -> LevelFilter {
    let rust_log = std::env::var("RUST_LOG").unwrap_or_default();

    let level = rust_log
        .split(',')
        .find_map(|entry| {
            entry
                .strip_prefix("hyperlight_guest=")
                .or_else(|| entry.strip_prefix("hyperlight_host="))
                .or_else(|| {
                    // Fallback: global log level (no '=' present)
                    if !entry.contains('=') {
                        Some(entry)
                    } else {
                        None
                    }
                })
        })
        .unwrap_or("");

    tracing::info!("Determined guest log level: {}", level);

    LevelFilter::from_str(level).unwrap_or(LevelFilter::ERROR)
}