Skip to content

Commit

Permalink
remove kernel limit for id mappings
Browse files Browse the repository at this point in the history
the limit depends on different kernel version
and not definitely required by spec

Signed-off-by: Ma Shimiao <[email protected]>
  • Loading branch information
Ma Shimiao committed Dec 2, 2017
1 parent efaa669 commit e7ee761
Showing 1 changed file with 0 additions and 4 deletions.
4 changes: 0 additions & 4 deletions validate/validate.go
Original file line number Diff line number Diff line change
Expand Up @@ -615,10 +615,6 @@ func (v *Validator) CheckLinux() (errs error) {

if (len(v.spec.Linux.UIDMappings) > 0 || len(v.spec.Linux.GIDMappings) > 0) && !nsTypeList[rspec.UserNamespace].newExist {
errs = multierror.Append(errs, errors.New("the UID/GID mappings requires a new User namespace to be specified as well"))
} else if len(v.spec.Linux.UIDMappings) > 5 {
errs = multierror.Append(errs, errors.New("only 5 UID mappings are allowed (linux kernel restriction)"))
} else if len(v.spec.Linux.GIDMappings) > 5 {
errs = multierror.Append(errs, errors.New("only 5 GID mappings are allowed (linux kernel restriction)"))
}

for k := range v.spec.Linux.Sysctl {
Expand Down

0 comments on commit e7ee761

Please sign in to comment.