-
Notifications
You must be signed in to change notification settings - Fork 9
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat(jstzd): docker container #559
base: leo/docker-pull-image
Are you sure you want to change the base?
Conversation
Codecov ReportAttention: Patch coverage is
Continue to review full report in Codecov by Sentry.
|
crates/jstzd/src/docker/container.rs
Outdated
pub async fn cleanup(&self) -> Result<()> { | ||
self.stop() | ||
.await | ||
.unwrap_or_else(|e| error!("Error stopping container: {}", e)); | ||
match self.client()?.remove_container(&self.id, None).await { | ||
Ok(_) => {} | ||
Err(e) => error!("Error removing container {}: {}", self.id, e), | ||
} | ||
Ok(()) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
c2c89f5
to
d418e66
Compare
baf27fb
to
d678a00
Compare
d418e66
to
f9b4724
Compare
d678a00
to
65c107e
Compare
f9b4724
to
934d878
Compare
65c107e
to
c06bb9a
Compare
934d878
to
df8eb4d
Compare
416a63e
to
f14da3a
Compare
f14da3a
to
2f1f863
Compare
Context
runnable_image create
container
Description
implement the create method on runnable image and docker container
Manually testing the PR
run
cargo test --package jstzd