Bug 1123454

Summary: [RFE] Document the proces of updating resource_limits.conf
Product: OpenShift Container Platform Reporter: Josep 'Pep' Turro Mauri <pep>
Component: DocumentationAssignee: Alex Dellapenta <adellape>
Status: CLOSED CURRENTRELEASE QA Contact: Bilhar <baulakh>
Severity: low Docs Contact:
Priority: medium    
Version: 2.1.0CC: jokerman, libra-onpremise-devel, mmccomas
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: Linux   
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2014-12-11 20:48:16 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---

Description Josep 'Pep' Turro Mauri 2014-07-25 17:35:50 UTC
Description of problem:
The header of the default /etc/openshift/resource_limits.conf file says:

# NOTE: If you change these, please run the following commands:
#       service mcollective restart
#       oo-cgroup-enable --with-all-containers
#       oo-pam-enable --with-all-containers
#       oo-admin-ctl-tc restart

but this is not documented elsewhere. The current guides don't document these commands (and oo-cgroup-enable in particular) AFAICT, nor the process of updating that file.

Version-Release number of selected component (if applicable):
OSE 2.1

How reproducible:

This request comes from a particular use case: if you try to increase memory_limit_in_bytes above your previousy set memory_memsw_limit_in_bytes, oo-cgroup-enable will fail; you have to increase the latter first.

Raising a doc bug to see if this could be officially documented better, either in the admin guide, in the file itself, or both.

Additional info:

For now the mentioned particular use case is documented in kbase (attached).

It seems it's not the first time this causes confusion, e.g. bug 1045540.

Comment 2 Alex Dellapenta 2014-12-11 20:48:16 UTC
This info was added to the OSE Administration Guide as part of the updates that went in for BZ#1133936: