Story #8343

[Crunch] cgroups root needs to be detected or configurable

Added by Brett Smith almost 6 years ago. Updated almost 6 years ago.

Status:
Resolved
Priority:
Normal
Assigned To:
Category:
Crunch
Target version:
Start date:
02/03/2016
Due date:
% Done:

100%

Estimated time:
(Total: 0.00 h)
Story points:
0.5

Description

crunch-job hardcodes the cgroups root (in a crunchstat switch) at /sys/fs/cgroup. CentOS 6 has this mount at /cgroup.

crunch-job should lets the administrator configure the cgroup root via CRUNCH_CGROUP_ROOT environment variable in the crunch-dispatch run script, just like we do with the docker/docker.io binary. If unset, use /sys/fs/cgroup.

Extend the install guide to document how to configure this.


Subtasks

Task #8594: add config knob to crunch-dispatchResolvedTom Clegg

Task #8358: review 8343-cgroup-rootResolvedTom Clegg

Associated revisions

Revision b0266435
Added by Tom Clegg almost 6 years ago

Merge branch '8343-cgroup-root' closes #8343

History

#1 Updated by Brett Smith almost 6 years ago

  • Description updated (diff)
  • Category set to Crunch
  • Assigned To set to Brett Smith
  • Target version set to 2016-02-17 Sprint
  • Story points set to 0.5

#2 Updated by Brett Smith almost 6 years ago

  • Assigned To deleted (Brett Smith)
  • Target version changed from 2016-02-17 Sprint to 2016-03-02 sprint

#3 Updated by Tom Clegg almost 6 years ago

  • Description updated (diff)

#4 Updated by Tom Clegg almost 6 years ago

  • Assigned To set to Tom Clegg

#5 Updated by Tom Clegg almost 6 years ago

  • Description updated (diff)
  • Status changed from New to In Progress

#6 Updated by Tom Clegg almost 6 years ago

8343-cgroup-root @ 6f3da61

#7 Updated by Brett Smith almost 6 years ago

Reviewing 6f3da61. Nice test.

  • There's a second place in crunch-job where -cgroup-root is hardcoded, under "# Non-docker run". Please edit this to use the new variable too.
  • In the runit script comment, please spell it "CentOS 6", since "centos6" is basically just an internal codename we use.

Go ahead and merge with those changes, since they're pretty straightforward. Thanks.

#8 Updated by Tom Clegg almost 6 years ago

  • Status changed from In Progress to Resolved
  • % Done changed from 50 to 100

Applied in changeset arvados|commit:b026643583d835a15d3baf6edef2df16ce678307.

Also available in: Atom PDF