Skip to content

plugin: Fix runtime interface documentation#1029

Merged
crosbymichael merged 1 commit intocontainerd:masterfrom
sameo:topic/typo
Jun 19, 2017
Merged

plugin: Fix runtime interface documentation#1029
crosbymichael merged 1 commit intocontainerd:masterfrom
sameo:topic/typo

Conversation

@sameo
Copy link
Copy Markdown

@sameo sameo commented Jun 19, 2017

Signed-off-by: Samuel Ortiz [email protected]

Copy link
Copy Markdown
Member

@estesp estesp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It looks like that whole section conflates container and task; probably due to the separation of those concepts after the initial implementation?

@codecov-io
Copy link
Copy Markdown

codecov-io commented Jun 19, 2017

Codecov Report

Merging #1029 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1029   +/-   ##
=======================================
  Coverage   59.28%   59.28%           
=======================================
  Files           5        5           
  Lines         781      781           
=======================================
  Hits          463      463           
  Misses        204      204           
  Partials      114      114

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7e3b7de...ddea395. Read the comment docs.

@sameo
Copy link
Copy Markdown
Author

sameo commented Jun 19, 2017

@estesp Sorry I missed that, fixed now.
I assume a Task is the initial execution context within a container ?

Copy link
Copy Markdown
Member

@estesp estesp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@crosbymichael
Copy link
Copy Markdown
Member

LGTM

@crosbymichael crosbymichael merged commit 100b93e into containerd:master Jun 19, 2017
ianlewis pushed a commit to ianlewis/containerd that referenced this pull request Dec 8, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants