Skip to content

Container limits - #1327

Merged
mattiaswal merged 4 commits into
mainfrom
container-limits
Dec 25, 2025
Merged

mattiaswal merged 4 commits into
mainfrom
container-limits

Conversation

@troglobit

Copy link
Copy Markdown
Contributor

Description

Add support for configurable CPU and memory limits on containers.

Checklist

Tick relevant boxes, this PR is-a or has-a:

  • Bugfix
    • Regression tests
    • ChangeLog updates (for next release)
  • Feature
    • YANG model change => revision updated?
    • Regression tests added?
    • ChangeLog updates (for next release)
    • Documentation added?
  • Test changes
    • Checked in changed Readme.adoc (make test-spec)
    • Added new test to group Readme.adoc and yaml file
  • Code style update (formatting, renaming)
  • Refactoring (please detail in commit messages)
  • Build related changes
  • Documentation content changes
    • ChangeLog updated (for major changes)
  • Other (please describe):

 - Improved cgroup support, including subgroup delegation.
 - Drop previously backported patch.

Signed-off-by: Joachim Wiberg <[email protected]>
Add resource-limit and resource-usage containers to YANG model.  Podman,
and later conmon, enforce CPU and memory limits in a delegated cgroupsv2
hierarchy managed by Finit.

Resource usage is queried from 'podman inspect', which has more nodes
than what is currently possible to limit.

Requires Finit 4.15, or later.

Signed-off-by: Joachim Wiberg <[email protected]>
@troglobit
troglobit requested a review from mattiaswal December 18, 2025 08:18

@mattiaswal mattiaswal left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Look sine 👍 just a minor comment

Comment thread src/confd/yang/confd/infix-containers.yang
@mattiaswal
mattiaswal merged commit fce59ef into main Dec 25, 2025
11 of 12 checks passed
@mattiaswal
mattiaswal deleted the container-limits branch December 25, 2025 20:55
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.

2 participants