Skip to content

Conversation

@cgwalters
Copy link
Collaborator

No description provided.

Copy link

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request introduces two significant debugging improvements for ephemeral VMs. First, it adds the ability to redirect virtiofsd output to log files within the container, which is now enabled by default for all mounts. Second, it sets up a systemd service within the guest VM to stream the system journal via a virtio-serial device to a log file in the container. The changes also include documentation for these new debugging features.

My review focuses on improving the asynchronous code quality. I've pointed out a couple of places where synchronous file I/O operations are used within async functions, which can block the executor thread. I've provided suggestions to use the asynchronous equivalents from tokio.

@cgwalters cgwalters force-pushed the ephemeral-debugging branch 2 times, most recently from 8d7ad6d to a5038fa Compare October 27, 2025 18:41
Add support for logging virtiofsd output to help diagnose filesystem
sharing issues between the container and VM.

Assisted-By: Claude Code (Sonnet 4.5)
Signed-off-by: Colin Walters <[email protected]>
Automatically stream the guest VM's systemd journal to the host for
easier debugging of boot failures and systemd service issues.

Assisted-By: Claude Code (Sonnet 4.5)
Signed-off-by: Colin Walters <[email protected]>
@cgwalters cgwalters force-pushed the ephemeral-debugging branch from a5038fa to 5c6aae1 Compare October 27, 2025 22:27
@cgwalters cgwalters enabled auto-merge (rebase) October 28, 2025 00:41
@cgwalters cgwalters merged commit 9094214 into bootc-dev:main Oct 28, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants