Uploaded image for project: 'OpenShift Virtualization'
  1. OpenShift Virtualization
  2. CNV-26658

Move cgroup logic to runc's repository

XMLWordPrintable

    • Icon: Epic Epic
    • Resolution: Unresolved
    • Icon: Undefined Undefined
    • None
    • None
    • CNV Virtualization
    • None
    • move-cgroup-logic-to-runc
    • False
    • Hide

      None

      Show
      None
    • False
    • To Do
    • 75% To Do, 25% In Progress, 0% Done

      OCP/Telco Definition of Done
      Epic Template descriptions and documentation.{}

      Epic Goal

      Runc have defined a "cgroup manager" [1] interface to help manage cgroups. This manager still lacks a lot of needed functionality, and is relatively new in general.

      In Kubevirt, we have our own cgroup manager [2] which serves as a thin middle-man between Kubevirt and Runc. However, since a lot of functionality lacks, a lot of the functionality is implemented in Kubevirt's manager instead of run.

      This epic is for moving the relevant code (that fits all cgroup users, and not necessarily related specifically to Kubevirt) away to runc. This will both help Kubevirt, since less code is maintained by us, and would also contribute to Kubernetes' community and echo-system.

       [1] https://github.com/opencontainers/runc/blob/v1.1.4/libcontainer/cgroups/cgroups.go#L7

       [2] https://github.com/kubevirt/kubevirt/blob/v0.59.0/pkg/virt-handler/cgroup/cgroup.go#L47

      Why is this important?

      This will both help Kubevirt, since less code is maintained by us, and would also contribute to Kubernetes' community and echo-system.

      Scenarios

      1. ...

      Acceptance Criteria

      • CI - MUST be running successfully with tests automated
      • Release Technical Enablement - Provide necessary release enablement details and documents.
      • ...

      Dependencies (internal and external)

      1. ...

      Previous Work (Optional):

      Open questions::

      Done Checklist

      • CI - CI is running, tests are automated and merged.
      • Release Enablement <link to Feature Enablement Presentation>
      • DEV - Upstream code and tests merged: <link to meaningful PR or GitHub Issue>
      • DEV - Upstream documentation merged: <link to meaningful PR or GitHub Issue>
      • DEV - Downstream build attached to advisory: <link to errata>
      • QE - Test plans in Polarion: <link or reference to Polarion>
      • QE - Automated tests merged: <link or reference to automated tests>
      • DOC - Downstream documentation merged: <link to meaningful PR>

              iholder@redhat.com Itamar Holder
              iholder@redhat.com Itamar Holder
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

                Created:
                Updated: