Skip to content
This repository was archived by the owner on May 12, 2021. It is now read-only.

virtcontainers: apply devices constraints#657

Merged
jodh-intel merged 1 commit into
kata-containers:masterfrom
devimc:topic/applyConstraints
Aug 30, 2018
Merged

virtcontainers: apply devices constraints#657
jodh-intel merged 1 commit into
kata-containers:masterfrom
devimc:topic/applyConstraints

Conversation

@devimc
Copy link
Copy Markdown

@devimc devimc commented Aug 28, 2018

Apply devices constraints to the container in the virtual machine

fixes #656

Signed-off-by: Julio Montes julio.montes@intel.com

@devimc
Copy link
Copy Markdown
Author

devimc commented Aug 28, 2018

Depends-on: kata-containers/packaging#157

@katacontainersbot
Copy link
Copy Markdown
Contributor

PSS Measurement:
Qemu: 167281 KB
Proxy: 4178 KB
Shim: 8812 KB

Memory inside container:
Total Memory: 2043464 KB
Free Memory: 2002580 KB

@codecov
Copy link
Copy Markdown

codecov Bot commented Aug 28, 2018

Codecov Report

❗ No coverage uploaded for pull request base (master@a303892). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #657   +/-   ##
=========================================
  Coverage          ?   65.34%           
=========================================
  Files             ?       85           
  Lines             ?     9878           
  Branches          ?        0           
=========================================
  Hits              ?     6455           
  Misses            ?     2766           
  Partials          ?      657

@opendev-zuul
Copy link
Copy Markdown

opendev-zuul Bot commented Aug 28, 2018

Build failed (third-party-check pipeline) integration testing with
OpenStack. For information on how to proceed, see
http://docs.openstack.org/infra/manual/developers.html#automated-testing

@jodh-intel
Copy link
Copy Markdown

jodh-intel commented Aug 29, 2018

lgtm

Approved with PullApprove

@devimc devimc force-pushed the topic/applyConstraints branch from 0530f4e to 286ee90 Compare August 29, 2018 14:59
@devimc devimc changed the title virtcontainers: apply blockIO and devices constraints virtcontainers: apply devices constraints Aug 29, 2018
Apply devices constraints to the container in the virtual machine

fixes kata-containers#656

Signed-off-by: Julio Montes <julio.montes@intel.com>
@devimc devimc force-pushed the topic/applyConstraints branch from 286ee90 to 137769a Compare August 29, 2018 15:15
@devimc
Copy link
Copy Markdown
Author

devimc commented Aug 29, 2018

pr updated, as we agreed in kata-containers/packaging#157, blockIO constraints should be applied in the host, not in the VM

@kata-containers/runtime

@amshinde
Copy link
Copy Markdown
Member

amshinde commented Aug 29, 2018

lgtm

Approved with PullApprove

@opendev-zuul
Copy link
Copy Markdown

opendev-zuul Bot commented Aug 29, 2018

Build failed (third-party-check pipeline) integration testing with
OpenStack. For information on how to proceed, see
http://docs.openstack.org/infra/manual/developers.html#automated-testing

@devimc
Copy link
Copy Markdown
Author

devimc commented Aug 29, 2018

the test for this change kata-containers/tests#666

@opendev-zuul
Copy link
Copy Markdown

opendev-zuul Bot commented Aug 29, 2018

Build failed (third-party-check pipeline) integration testing with
OpenStack. For information on how to proceed, see
http://docs.openstack.org/infra/manual/developers.html#automated-testing

@katacontainersbot
Copy link
Copy Markdown
Contributor

PSS Measurement:
Qemu: 167632 KB
Proxy: 4278 KB
Shim: 8752 KB

Memory inside container:
Total Memory: 2043464 KB
Free Memory: 2003564 KB

@jodh-intel
Copy link
Copy Markdown

Merging!

@jodh-intel jodh-intel merged commit 7d14aea into kata-containers:master Aug 30, 2018
@egernst egernst removed the review label Aug 30, 2018
jcvenegas pushed a commit to jcvenegas/runtime that referenced this pull request Aug 30, 2018
…ints

virtcontainers: apply devices constraints
GabyCT added a commit to GabyCT/tests-1 that referenced this pull request Aug 30, 2018
Now that kata-containers/runtime#657 has been solved, the device passed as
volume should also needs to be passed as device.

Fixes kata-containers#684

Signed-off-by: Gabriela Cervantes <gabriela.cervantes.tellez@intel.com>
@sboeuf sboeuf added the enhancement Improvement to an existing feature label Sep 12, 2018
egernst pushed a commit to egernst/runtime that referenced this pull request Feb 9, 2021
Includes significant fixes including change to allow proc
mount only if it is rootfs.

Shortlog:

d736ef14 (tag: v1.0.0-rc9) VERSION: update to 1.0.0-rc9
d463f648 *: verify that operations on /proc/... are on procfs
9aef5044 vendor: update github.com/opencontainers/selinux
84373aaa Add SCMP_ACT_LOG as a valid Seccomp action (kata-containers#1951)
331692ba Only allow proc mount if it is procfs
af7b6547 libcontainer/nsenter: Don't import C in non-cgo file
718a566e cgroup: support mount of cgroup2
115d4b9e bump golang/protobuf v1.0.0
85c02f3f bump coreos/go-systemd v19, godbus/dbus v5.0.1
21498b8e bump mrunalp/fileutils 7d4729fb36185a7c1719923406c9d40e54fb93c7
eb86f603 bump syndtr/gocapability
d98352740cb2c55f81556b63d4a1ec64c5a319c2
1150ce9c bump urfave/cli v1.20.0
8e4f645f bump docker/go-units v0.3.3
0fc06623 bump cyphar/filepath-securejoin v0.2.2
414a39de bump containerd/console
0650fd9eeb50bab4fc99dceb9f2e14cf58f36e7f
de24d733 bump github.com/pkg/errors 0.8.1
4be3c48e Reformat vendor.conf and pin all deps by git-sha
524cb7c3 libcontainer: add systemd.UnifiedManager
ec111368 libcontainer, cgroups: rename systemd.Manager to LegacyManager
1932917b libcontainer: add initial support for cgroups v2
4316e4d0 Bump x/sys and update syscall to start Risc-V support
0bc069d7 nsenter: fix clang-tidy warning
b225ef58 nsenter: minor clean up
e4aa7342 Rename cgroups_windows.go to cgroups_unsupported.go
c740965a libcontainer: update masked paths of /proc
518c8558 Remove libcontainer detection for systemd features
4ca00773 Update vendored dependencies to remove go-systemd/util
588f040a Avoid the dependency on cgo through go-systemd/util package
afc24792 Make get devices function public
9c822e48 cgroups/fs: check nil pointers in cgroup manager
1712af0e man: fix man-pages
f08cdaee Skip searching /dev/.udev for device nodes.
808e809f doc: First process in container needs `Init: true`
351bfb4b integration: remove blkio.weight (unavailable in kernel 5.0)
7e678625 Bump CRIU to 3.12
68cc1a77 Update busybox source and fix runc exec bug
371d13c9 Update bash completion for v1.0.0 release
7a9ffa89 Change the permissions of the notify listener socket to rwx for
everyone
e7831f2a Update to Go 1.12 and drop obsolete versions
472fe623 criu image path permission error in rootless checkpoint

Fixes kata-containers#657

Signed-off-by: Archana Shinde <archana.m.shinde@intel.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

enhancement Improvement to an existing feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Support devices constraints

6 participants