Delete outdated Phase 1 implementation summary, apply minor improvements to guest-agent reboot logic, and enhance vsock availability checks in E2E tests.

Author: dpsoft

PHASE1_IMPLEMENTATION.md

@@ -602,7 +602,7 @@ fn handle_connection(fd: RawFd) -> Result<(), String> {
                 // Shutdown
                 eprintln!("Shutdown requested");
                 unsafe {
-                    libc::reboot(libc::LINUX_REBOOT_CMD_POWER_OFF as i32);
+                    libc::reboot(libc::LINUX_REBOOT_CMD_POWER_OFF);
                 }
                 return Ok(());
             }
@@ -1223,8 +1223,7 @@ fn read_netdev() -> (u64, u64) {
     };
     for line in content.lines() {
         let trimmed = line.trim();
-        if trimmed.starts_with("eth0:") {
-            let after_colon = &trimmed["eth0:".len()..];
+        if let Some(after_colon) = trimmed.strip_prefix("eth0:") {
             let fields: Vec<&str> = after_colon.split_whitespace().collect();
             let rx = fields
                 .first()

guest-agent/src/main.rs

@@ -38,7 +38,17 @@ fn kvm_available() -> bool {
 }
 
 fn vsock_available() -> bool {
-    std::path::Path::new("/dev/vhost-vsock").exists()
+    let path = std::path::Path::new("/dev/vhost-vsock");
+    // Check existence and that we can actually open it (not just EACCES)
+    match std::fs::File::open(path) {
+        Ok(_) => true,
+        Err(e) if e.kind() == std::io::ErrorKind::NotFound => false,
+        Err(e) if e.raw_os_error() == Some(libc::EACCES) => {
+            eprintln!("skipping: /dev/vhost-vsock exists but is not accessible: {e}");
+            false
+        }
+        Err(_) => false,
+    }
 }
 
 fn kvm_artifacts_from_env() -> Option<(PathBuf, Option<PathBuf>)> {