plumb max_guest_log_level through MultiUseSandbox::from_snapshot

[sethryanrollins]

draft for #1698

[jsturtevant]

this looks in the right direction, @sethryanrollins could you sign the DCO or fix the commits and rebase? then we can run CI

[ludfjig]

thanks for your contribution and you interest in hyperlight @sethryanrollins! What I was hoping to do is to address the issue for regular snapshots (taken after init has ran), rather than for these pre-init snapshots (which are not public anyway). It looks like the current state of the PR only addresses pre-init snapshot. Would you be willing to rewrite the PR to deal with post-init snapshots instead? Please let me know if you have any questions as it probably won't be trivial!

[yoshuawuyts]

#1725 was merged yesterday, renaming max_guest_log_level to guest_log_level. Given we're introducing a new public API in this PR here, we may want to update the API name here to match. (cc/ @jprendes)

[dblnz]

LGTM!

[jprendes]

we may want to update the API name here to match

I think this is fine, as we are going to mark the SandboxConfig as deprecated. It will stay as an internal implementation detail.
Currently the SandboxBuilder will build if you set the log level on a snapshot, this will let us remove that failure mode, nice!

[jprendes]

Rebased the changes onto latest main.