Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Update spec to master, switch to int64 for memory limits #1495

Merged
merged 3 commits into from
Jun 27, 2017
Merged

Update spec to master, switch to int64 for memory limits #1495

merged 3 commits into from
Jun 27, 2017

Commits on Jun 27, 2017

  1. Update to runtime spec 198f23f 

    Updates memory limits to be int64, and removes Platform from spec.

Signed-off-by: Justin Cormack <justin.cormack@docker.com>
    @justincormack
    justincormack committed Jun 27, 2017
    Configuration menu
    Copy the full SHA
    e8ef602 View commit details
    Browse the repository at this point in the history

  2. Remove Platform as no longer in OCI spec 

    This was never used, just validated, so was removed from spec.

Signed-off-by: Justin Cormack <justin.cormack@docker.com>
    @justincormack
    justincormack committed Jun 27, 2017
    Configuration menu
    Copy the full SHA
    e114618 View commit details
    Browse the repository at this point in the history

  3. Update memory specs to use int64 not uint64 

    replace opencontainers#1492 opencontainers#1494
fix opencontainers#1422

Since opencontainers/runtime-spec#876 the memory
specifications are now `int64`, as that better matches the visible interface where
`-1` is a valid value. Otherwise finding the correct value was difficult as it
was kernel dependent.

Signed-off-by: Justin Cormack <justin.cormack@docker.com>
    @justincormack
    justincormack committed Jun 27, 2017
    Configuration menu
    Copy the full SHA
    3d9074e View commit details
    Browse the repository at this point in the history