process_info should be ProcessDetails, otherwise it can't be parsed

containerd · Sep 3, 2024 · 71de1b4 · 71de1b4
1 parent 0911c1c
commit 71de1b4
Showing 1 changed file with 12 additions and 2 deletions.
diff --git a/crates/runc-shim/src/container.rs b/crates/runc-shim/src/container.rs
@@ -22,7 +22,8 @@ use containerd_shim::{
     protos::{
         api::{CreateTaskRequest, ExecProcessRequest, ProcessInfo, StateResponse},
         cgroups::metrics::Metrics,
-        protobuf::{well_known_types::any::Any, Message},
+        protobuf::{well_known_types::any::Any, Message, MessageDyn},
+        shim::oci::ProcessDetails,
     },
     Error,
 };
@@ -190,7 +191,16 @@ where
         for process_info in &mut processes_info {
             for (exec_id, process) in &self.processes {
                 if process_info.pid as i32 == process.pid().await {
-                    process_info.set_info(Any::parse_from_bytes(exec_id.as_bytes())?);
+                    let process_details = ProcessDetails {
+                        exec_id: exec_id.to_string(),
+                        special_fields: Default::default(),
+                    };
+                    let v = Any {
+                        type_url: process_details.descriptor_dyn().full_name().to_string(),
+                        value: process_details.write_to_bytes()?,
+                        special_fields: Default::default(),
+                    };
+                    process_info.set_info(v);
                     break;
                 }
             }