Project

General

Profile

Actions

Idea #8343

closed

[Crunch] cgroups root needs to be detected or configurable

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

Status:
Resolved
Priority:
Normal
Assigned To:
Category:
Crunch
Target version:
Start date:
02/03/2016
Due date:
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 2 (0 open2 closed)

Task #8594: add config knob to crunch-dispatchResolvedTom Clegg02/03/2016Actions
Task #8358: review 8343-cgroup-rootResolvedTom Clegg02/03/2016Actions
Actions #1

Updated by Brett Smith almost 9 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
Actions #2

Updated by Brett Smith almost 9 years ago

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

Updated by Tom Clegg almost 9 years ago

  • Description updated (diff)
Actions #4

Updated by Tom Clegg almost 9 years ago

  • Assigned To set to Tom Clegg
Actions #5

Updated by Tom Clegg almost 9 years ago

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

Updated by Tom Clegg almost 9 years ago

8343-cgroup-root @ 6f3da61

Actions #7

Updated by Brett Smith almost 9 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.

Actions #8

Updated by Tom Clegg almost 9 years ago

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

Applied in changeset arvados|commit:b026643583d835a15d3baf6edef2df16ce678307.

Actions

Also available in: Atom PDF