Skip to content
This repository was archived by the owner on Feb 8, 2021. It is now read-only.

Conversation

@tvelocity
Copy link
Contributor

This PR implements the delete command as defined by the OCI spec.

To properly separate the container stopped state, from a deleted (non existent state) we add the necessary container states to runv.

Track the OCI container states:
- creating
- created
- running
- stopped

Signed-off-by: Antonios Motakis <antonios.motakis@huawei.com>
@tvelocity
Copy link
Contributor Author

Currently looking into the failing tests.

The list and state commands of the runv OCI interface always
return "running" as the status of the container. Return the
right status of each container via the containerd API.

Signed-off-by: Antonios Motakis <antonios.motakis@huawei.com>
Currently runv automatically destroys any container that
finish executing. The OCI spec defines that the container
shall enter a stopped status instead; an explicit delete command
is defined for cleaning up any container resources.

Don't delete the container automatically on exit, and add an
explicit delete command to the OCI interface of runv. To get
around the limited containerd interface of runv, use a special
status for containers that are about to be deleted.

Signed-off-by: Antonios Motakis <antonios.motakis@huawei.com>
@WeiZhang555
Copy link
Contributor

ping @laijs Can you take a look at this? ( @tvelocity is our guy from Europe 😄 )

@gnawux
Copy link
Member

gnawux commented Aug 3, 2017

@tvelocity we merged #537 , which aimed to improve the compatibility with the latest containerd and changed the cli much. Could you help check/update this PR? Thank you very much.

@laijs
Copy link
Contributor

laijs commented Aug 3, 2017

@tvelocity runv cli is being improved continuously and had just been refactored at #537 . It seams that the changes in this pr is outdated.

Moreover, runv delete was also implemented in #537. Feel free to test it and give any possible feedback.

However, runv list & runv state are changed less, and the 'container status' shown by these two commands is still always "running". Could you create new PR for it please if any possible? And this one is closed.

Thank you for your contribution.

@laijs laijs closed this Aug 3, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants