Skip to content

Fix example using old GenerateSpec function#1549

Merged
crosbymichael merged 1 commit intocontainerd:masterfrom
estesp:fix-getting-started-examples
Sep 22, 2017
Merged

Fix example using old GenerateSpec function#1549
crosbymichael merged 1 commit intocontainerd:masterfrom
estesp:fix-getting-started-examples

Conversation

@estesp
Copy link
Copy Markdown
Member

@estesp estesp commented Sep 22, 2017

This fixes the example in the getting started guide. The full example
was already correct; tested compilation and proper runtime on latest
master.

Signed-off-by: Phil Estes [email protected]

Fixes: #1521 thanks to @rutsky for finding/solving

This fixes the example in the getting started guide. The full example
was already correct; tested compilation and proper runtime on latest
master.

Signed-off-by: Phil Estes <[email protected]>
@crosbymichael
Copy link
Copy Markdown
Member

crosbymichael commented Sep 22, 2017

LGTM

@crosbymichael crosbymichael merged commit 316ce18 into containerd:master Sep 22, 2017
@codecov-io
Copy link
Copy Markdown

Codecov Report

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

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1549   +/-   ##
=======================================
  Coverage   42.36%   42.36%           
=======================================
  Files          24       24           
  Lines        3368     3368           
=======================================
  Hits         1427     1427           
  Misses       1612     1612           
  Partials      329      329

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 9a7e776...fe1e00c. Read the comment docs.

@stevvooe
Copy link
Copy Markdown
Member

@estesp Thanks!

mauriciovasquezbernal pushed a commit to kinvolk/containerd that referenced this pull request Nov 13, 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.

Examples in Getting started guide outdated (won't compile with containerd 1.0.0-beta.0)

4 participants