Skip to content

Skip AdditionalGID's for LCOW oci spec#2698

Merged
dmcgowan merged 1 commit intocontainerd:masterfrom
jterry75:lcow_spec_opt
Oct 3, 2018
Merged

Skip AdditionalGID's for LCOW oci spec#2698
dmcgowan merged 1 commit intocontainerd:masterfrom
jterry75:lcow_spec_opt

Conversation

@jterry75
Copy link
Copy Markdown
Contributor

@jterry75 jterry75 commented Oct 3, 2018

Signed-off-by: Justin Terry (VM) [email protected]

@jterry75
Copy link
Copy Markdown
Contributor Author

jterry75 commented Oct 3, 2018

This commit fixes the ability to run LCOW via CTR.

C:\> ctr image pull --snapshotter windows-lcow --platform linux/amd64 docker.io/library/alpine:latest
C:\> ctr run --runtime io.containerd.runhcs.v1 --snapshotter windows-lcow --rm docker.io/library/alpine:latest lcow-test sh -c "echo Hello World!"
Hello World!

@codecov-io
Copy link
Copy Markdown

Codecov Report

Merging #2698 into master will decrease coverage by 0.01%.
The diff coverage is 0%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2698      +/-   ##
==========================================
- Coverage    43.6%   43.59%   -0.02%     
==========================================
  Files         101      101              
  Lines       10718    10722       +4     
==========================================
  Hits         4674     4674              
- Misses       5310     5314       +4     
  Partials      734      734
Flag Coverage Δ
#linux 47.25% <0%> (-0.04%) ⬇️
#windows 40.72% <0%> (-0.02%) ⬇️
Impacted Files Coverage Δ
oci/spec_opts.go 20.77% <0%> (-0.12%) ⬇️

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 de4bb2d...223acbc. Read the comment docs.

@crosbymichael
Copy link
Copy Markdown
Member

LGTM

1 similar comment
@dmcgowan
Copy link
Copy Markdown
Member

dmcgowan commented Oct 3, 2018

LGTM

@dmcgowan dmcgowan added this to the 1.2 milestone Oct 3, 2018
@dmcgowan dmcgowan merged commit 0e6a562 into containerd:master Oct 3, 2018
@jterry75 jterry75 deleted the lcow_spec_opt branch October 3, 2018 21:00
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