diff options
author | Mohit Agrawal <moagrawal@redhat.com> | 2019-10-22 18:52:25 +0530 |
---|---|---|
committer | hari gowtham <hari.gowtham005@gmail.com> | 2019-11-14 13:17:41 +0000 |
commit | a6c1a84b47110ec9e72f25db7ed5af9de64d3ff4 (patch) | |
tree | dc8a6a180762678a88574883d1abce25ca561742 | |
parent | 7808a10af0949ee19ba4c0ab000732ba7eb67f02 (diff) |
extras: Cgroup(CPU/Mem) restriction are not working on gluster process
Problem: After Configure the Cgroup(CPU/MEM) limit to a gluster processes
resource(CPU/MEM) limits are not applicable to the gluster
processes.Cgroup limits are not applicable because all threads are
not moved into a newly created cgroup to apply restriction.
Solution: To move a gluster thread to newly created cgroup change the
condition in script
> Change-Id: I8ad81c69200e4ec43a74f6052481551cf835354c
> Fixes: bz#1764208
> Signed-off-by: Mohit Agrawal <moagrawal@redhat.com>
> (cherry picked from commit f5811979935ce607391825ac6913a95f588818e3)
> (Reviewed on upstream link https://review.gluster.org/#/c/glusterfs/+/23599/)
Change-Id: I8ad81c69200e4ec43a74f6052481551cf835354c
Fixes: bz#1766424
Signed-off-by: Mohit Agrawal <moagrawal@redhat.com>
-rwxr-xr-x | extras/control-cpu-load.sh | 2 | ||||
-rwxr-xr-x | extras/control-mem.sh | 2 |
2 files changed, 2 insertions, 2 deletions
diff --git a/extras/control-cpu-load.sh b/extras/control-cpu-load.sh index b739c821055..52dcf62fd9f 100755 --- a/extras/control-cpu-load.sh +++ b/extras/control-cpu-load.sh @@ -104,7 +104,7 @@ echo "Setting $quota_value to cpu.cfs_quota_us for gluster_cgroup." echo ${quota_value} > ${LOC}/${cgroup_name}/cpu.cfs_quota_us if ps -T -p ${daemon_pid} | grep gluster > /dev/null; then - for thid in `ps -T -p ${daemon_pid} | grep gluster | awk -F " " '{print $2}'`; + for thid in `ps -T -p ${daemon_pid} | grep -v SPID | awk -F " " '{print $2}'`; do echo ${thid} > ${LOC}/${cgroup_name}/tasks ; done diff --git a/extras/control-mem.sh b/extras/control-mem.sh index 38aa2a08748..91b36f8107a 100755 --- a/extras/control-mem.sh +++ b/extras/control-mem.sh @@ -116,7 +116,7 @@ else fi if ps -T -p ${daemon_pid} | grep gluster > /dev/null; then - for thid in `ps -T -p ${daemon_pid} | grep gluster | awk -F " " '{print $2}'`; + for thid in `ps -T -p ${daemon_pid} | grep -v SPID | awk -F " " '{print $2}'`; do echo ${thid} > ${LOC}/${cgroup_name}/tasks ; done |