Skip to content

gitaly: Expose max_cgroups_per_repo configuration

James Liu requested to merge jliu/add-max-cgroups-per-repo into master

What does this MR do?

gitaly: Expose max_cgroups_per_repo configuration

A new cgroups configuration parameter called max_cgroups_per_repo was introduced with gitlab-org/gitaly#5689 (closed). This parameter allows independent Git processes targeting a single repository to be distributed across multiple repository cgroups. This enables stricter limits to be defined for repository cgroups while still allowing for bursty workloads comprising of independent Git operations against a single repo.

Expose this parameter in the Gitaly Helm Chart. Purposely exclude this from the example values.yaml snippet as we haven't dogfooded this option yet.

Related issues

gitlab-org/gitaly#6397 (closed)

Companion Omnibus MR: gitlab-org/omnibus-gitlab!7952 (merged)

Author checklist

For general guidance, please follow our Contributing guide.

Required

For anything in this list which will not be completed, please provide a reason in the MR discussion.

  • Merge Request Title and Description are up to date, accurate, and descriptive.
  • MR targeting the appropriate branch.
  • MR has a green pipeline.
  • Documentation created/updated.
  • Tests added/updated, and test plan for scenarios not covered by automated tests.
  • Equivalent MR/issue for omnibus-gitlab opened.

Reviewers checklist

Edited by Clemens Beck

Merge request reports