Skip to content

Commit

Permalink
shim: Implement the pause and resume interfaces
Browse files Browse the repository at this point in the history
  • Loading branch information
zzzzzzzzzy9 committed Jul 4, 2024
1 parent 666a5ec commit e1a753c
Show file tree
Hide file tree
Showing 4 changed files with 100 additions and 0 deletions.
10 changes: 10 additions & 0 deletions crates/runc-shim/src/container.rs
Original file line number Diff line number Diff line change
Expand Up @@ -54,6 +54,8 @@ pub trait Container {
async fn stats(&self) -> Result<Metrics>;
async fn all_processes(&self) -> Result<Vec<ProcessInfo>>;
async fn close_io(&mut self, exec_id: Option<&str>) -> Result<()>;
async fn pause(&mut self) -> Result<()>;
async fn resume(&mut self) -> Result<()>;
}

#[async_trait]
Expand Down Expand Up @@ -192,6 +194,14 @@ where
let process = self.get_mut_process(exec_id)?;
process.close_io().await
}

async fn pause(&mut self) -> Result<()> {
self.init.pause().await
}

async fn resume(&mut self) -> Result<()> {
self.init.resume().await
}
}

impl<T, E, P> ContainerTemplate<T, E, P>
Expand Down
12 changes: 12 additions & 0 deletions crates/runc-shim/src/processes.rs
Original file line number Diff line number Diff line change
Expand Up @@ -55,6 +55,8 @@ pub trait Process {
async fn stats(&self) -> Result<Metrics>;
async fn ps(&self) -> Result<Vec<ProcessInfo>>;
async fn close_io(&mut self) -> Result<()>;
async fn pause(&mut self) -> Result<()>;
async fn resume(&mut self) -> Result<()>;
}

#[async_trait]
Expand All @@ -65,6 +67,8 @@ pub trait ProcessLifecycle<P: Process> {
async fn update(&self, p: &mut P, resources: &LinuxResources) -> Result<()>;
async fn stats(&self, p: &P) -> Result<Metrics>;
async fn ps(&self, p: &P) -> Result<Vec<ProcessInfo>>;
async fn pause(&self, p: &mut P) -> Result<()>;
async fn resume(&self, p: &mut P) -> Result<()>;
}

pub struct ProcessTemplate<S> {
Expand Down Expand Up @@ -198,4 +202,12 @@ where
}
Ok(())
}

async fn pause(&mut self) -> Result<()> {
self.lifecycle.clone().pause(self).await
}

async fn resume(&mut self) -> Result<()> {
self.lifecycle.clone().resume(self).await
}
}
48 changes: 48 additions & 0 deletions crates/runc-shim/src/runc.rs
Original file line number Diff line number Diff line change
Expand Up @@ -349,6 +349,46 @@ impl ProcessLifecycle<InitProcess> for RuncInitLifecycle {
})
.collect())
}

#[cfg(target_os = "linux")]
async fn pause(&self, p: &mut InitProcess) -> Result<()> {
match p.state {
Status::RUNNING => {
p.state = Status::PAUSING;
if let Err(e) = self.runtime.pause(p.id.as_str()).await {
p.state = Status::RUNNING;
return Err(runtime_error(&self.bundle, e, "OCI runtime pause failed").await);
}
p.state = Status::PAUSED;
Ok(())
}
_ => Err(other!("cannot pause when in {:?} state", p.state)),
}
}

#[cfg(not(target_os = "linux"))]
async fn pause(&self, _p: &mut InitProcess) -> Result<()> {
Err(Error::Unimplemented("pause".to_string()))
}

#[cfg(target_os = "linux")]
async fn resume(&self, p: &mut InitProcess) -> Result<()> {
match p.state {
Status::PAUSED => {
if let Err(e) = self.runtime.resume(p.id.as_str()).await {
return Err(runtime_error(&self.bundle, e, "OCI runtime pause failed").await);
}
p.state = Status::RUNNING;
Ok(())
}
_ => Err(other!("cannot resume when in {:?} state", p.state)),
}
}

#[cfg(not(target_os = "linux"))]
async fn resume(&self, _p: &mut InitProcess) -> Result<()> {
Err(Error::Unimplemented("resume".to_string()))
}
}

impl RuncInitLifecycle {
Expand Down Expand Up @@ -473,6 +513,14 @@ impl ProcessLifecycle<ExecProcess> for RuncExecLifecycle {
async fn ps(&self, _p: &ExecProcess) -> Result<Vec<ProcessInfo>> {
Err(Error::Unimplemented("exec ps".to_string()))
}

async fn pause(&self, _p: &mut ExecProcess) -> Result<()> {
Err(Error::Unimplemented("exec pause".to_string()))
}

async fn resume(&self, _p: &mut ExecProcess) -> Result<()> {
Err(Error::Unimplemented("exec resume".to_string()))
}
}

async fn copy_console(
Expand Down
30 changes: 30 additions & 0 deletions crates/runc-shim/src/task.rs
Original file line number Diff line number Diff line change
Expand Up @@ -50,6 +50,10 @@ use std::path::Path;

#[cfg(target_os = "linux")]
use cgroups_rs::hierarchies::is_cgroup2_unified_mode;
use containerd_shim::{
api::{PauseRequest, ResumeRequest},
protos::events::task::{TaskPaused, TaskResumed},
};
#[cfg(target_os = "linux")]
use containerd_shim::{
error::{Error, Result},
Expand Down Expand Up @@ -288,6 +292,32 @@ where
})
}

async fn pause(&self, _ctx: &TtrpcContext, req: PauseRequest) -> TtrpcResult<Empty> {
info!("pause request for {:?}", req);
let mut container = self.get_container(req.id()).await?;
container.pause().await?;
self.send_event(TaskPaused {
container_id: req.id.to_string(),
..Default::default()
})
.await;
info!("pause request for {:?} returns successfully", req);
Ok(Empty::new())
}

async fn resume(&self, _ctx: &TtrpcContext, req: ResumeRequest) -> TtrpcResult<Empty> {
info!("resume request for {:?}", req);
let mut container = self.get_container(req.id()).await?;
container.resume().await?;
self.send_event(TaskResumed {
container_id: req.id.to_string(),
..Default::default()
})
.await;
info!("resume request for {:?} returns successfully", req);
Ok(Empty::new())
}

async fn kill(&self, _ctx: &TtrpcContext, req: KillRequest) -> TtrpcResult<Empty> {
info!("Kill request for {:?}", req);
let mut container = self.get_container(req.id()).await?;
Expand Down

0 comments on commit e1a753c

Please sign in to comment.