Skip to content

[release/1.0] fix running a container with config.User#2005

Merged
estesp merged 4 commits intocontainerd:release/1.0from
AkihiroSuda:fix-user-1.0
Jan 16, 2018
Merged

[release/1.0] fix running a container with config.User#2005
estesp merged 4 commits intocontainerd:release/1.0from
AkihiroSuda:fix-user-1.0

Conversation

@AkihiroSuda
Copy link
Copy Markdown
Member

Cherry-pick: #1994

@codecov-io
Copy link
Copy Markdown

Codecov Report

Merging #2005 into release/1.0 will increase coverage by 0.02%.
The diff coverage is 0%.

Impacted file tree graph

@@               Coverage Diff               @@
##           release/1.0    #2005      +/-   ##
===============================================
+ Coverage        50.57%   50.59%   +0.02%     
===============================================
  Files               81       81              
  Lines             7140     7137       -3     
===============================================
  Hits              3611     3611              
+ Misses            2838     2835       -3     
  Partials           691      691
Flag Coverage Δ
#linux 50.59% <0%> (+0.02%) ⬆️
Impacted Files Coverage Δ
oci/spec_opts_unix.go 5.82% <0%> (+0.07%) ⬆️

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 0a2c2a2...2eb4408. Read the comment docs.

@crosbymichael
Copy link
Copy Markdown
Member

LGTM

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

@estesp estesp merged commit dafb45d into containerd:release/1.0 Jan 16, 2018
@stevvooe
Copy link
Copy Markdown
Member

I just wanted to note that we don't typically backport bugs in ctr. We'll make an exception this time, but ctr is not a supported component.

@stevvooe
Copy link
Copy Markdown
Member

Also, we had said here that this would not go into 1.0.1.

@stevvooe stevvooe added this to the 1.0.1 milestone Jan 16, 2018
@stevvooe
Copy link
Copy Markdown
Member

Also, this didn't need 4 commits.

@AkihiroSuda
Copy link
Copy Markdown
Member Author

@stevvooe sorry 😅 I'll try to be more careful next time

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