Skip to content

[release/1.1] Backport: With-helper for supplemental gid support#2637

Merged
dmcgowan merged 1 commit intocontainerd:release/1.1from
estesp:cherrypick-supplemental-grps
Sep 12, 2018
Merged

[release/1.1] Backport: With-helper for supplemental gid support#2637
dmcgowan merged 1 commit intocontainerd:release/1.1from
estesp:cherrypick-supplemental-grps

Conversation

@estesp
Copy link
Copy Markdown
Member

@estesp estesp commented Sep 12, 2018

[Backported to release/1.1]
Can be used to conform to Docker engine behavior of adding additional
GIDs to spec when found in /etc/group of image filesystem

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

[Backported to release/1.1]
Can be used to conform to Docker engine behavior of adding additional
GIDs to spec when found in /etc/group of image filesystem

Signed-off-by: Phil Estes <[email protected]>
@estesp estesp force-pushed the cherrypick-supplemental-grps branch from 821ba91 to 19735b5 Compare September 12, 2018 15:32
@codecov-io
Copy link
Copy Markdown

Codecov Report

Merging #2637 into release/1.1 will decrease coverage by 0.32%.
The diff coverage is 0%.

Impacted file tree graph

@@               Coverage Diff               @@
##           release/1.1    #2637      +/-   ##
===============================================
- Coverage        49.27%   48.94%   -0.33%     
===============================================
  Files               85       85              
  Lines             7546     7596      +50     
===============================================
  Hits              3718     3718              
- Misses            3150     3200      +50     
  Partials           678      678
Flag Coverage Δ
#linux 48.94% <0%> (-0.33%) ⬇️
Impacted Files Coverage Δ
oci/spec_opts_unix.go 19.04% <0%> (-2.73%) ⬇️

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 db009b3...19735b5. Read the comment docs.

@crosbymichael
Copy link
Copy Markdown
Member

LGTM

1 similar comment
@dmcgowan
Copy link
Copy Markdown
Member

LGTM

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