Skip to content

snapshot: use switch to parse snapshot.Kind#1463

Merged
dmcgowan merged 1 commit intocontainerd:masterfrom
stevvooe:switch-snapshot-kind
Sep 5, 2017
Merged

snapshot: use switch to parse snapshot.Kind#1463
dmcgowan merged 1 commit intocontainerd:masterfrom
stevvooe:switch-snapshot-kind

Conversation

@stevvooe
Copy link
Copy Markdown
Member

@stevvooe stevvooe commented Sep 1, 2017

Signed-off-by: Stephen J Day [email protected]

@AkihiroSuda PTAL

@stevvooe stevvooe added this to the containerd beta milestone Sep 1, 2017
@codecov-io
Copy link
Copy Markdown

codecov-io commented Sep 1, 2017

Codecov Report

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

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1463   +/-   ##
=======================================
  Coverage   40.82%   40.82%           
=======================================
  Files          23       23           
  Lines        2920     2920           
=======================================
  Hits         1192     1192           
  Misses       1451     1451           
  Partials      277      277

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 f4b8498...1c31199. Read the comment docs.

Comment thread snapshot/snapshotter.go Outdated
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

The casing has changed here

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Na, we just go with this casing then flatten the case when parsing.

Comment thread snapshot/snapshotter.go Outdated
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

typo: Unkown

Comment thread snapshot/snapshotter.go Outdated
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

unknown -> Unknown

@stevvooe stevvooe force-pushed the switch-snapshot-kind branch from bd77bc7 to 1c31199 Compare September 5, 2017 18:04
@crosbymichael
Copy link
Copy Markdown
Member

LGTM

1 similar comment
@dmcgowan
Copy link
Copy Markdown
Member

dmcgowan commented Sep 5, 2017

LGTM

@dmcgowan dmcgowan merged commit 887f97f into containerd:master Sep 5, 2017
@stevvooe stevvooe deleted the switch-snapshot-kind branch September 5, 2017 21:15
mauriciovasquezbernal pushed a commit to kinvolk/containerd that referenced this pull request Nov 13, 2020
…8s.io/kubernetes

Remove dependencies on k8s.io/kubernetes
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.

5 participants