Skip to content

feat: support adjusting child proc oom scores #12655

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 10 commits into from
Apr 3, 2024
Merged

feat: support adjusting child proc oom scores #12655

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
02eb747
fix: adjust process oom_score_adj to 0
sreya Mar 19, 2024
b3a787a
fmt
sreya Mar 19, 2024
f37242b
oom score
sreya Mar 20, 2024
403a637
fmt
sreya Mar 20, 2024
a214974
lint
sreya Mar 21, 2024
54b7f4c
Merge branch 'main' into jon/procprio
sreya Apr 2, 2024
17714ed
debounce logs
sreya Apr 2, 2024
561279b
lint
sreya Apr 2, 2024
2ddeb8e
fmt
sreya Apr 2, 2024
9f5f63c
pass through env var
sreya Apr 3, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
fmt
  • Loading branch information
@sreya
sreya committed Mar 20, 2024
commit 403a63719d20b36ea156f5b48121d3c3c94914b5
6 changes: 4 additions & 2 deletions agent/agent.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -62,8 +62,10 @@ const (

// EnvProcPrioMgmt determines whether we attempt to manage
// process CPU and OOM Killer priority.
const EnvProcPrioMgmt = "CODER_PROC_PRIO_MGMT"
const EnvProcOOMScore = "CODER_PROC_OOM_SCORE"
const (
EnvProcPrioMgmt = "CODER_PROC_PRIO_MGMT"
EnvProcOOMScore = "CODER_PROC_OOM_SCORE"
)

type Options struct {
Filesystem afero.Fs
Expand Down
1 change: 0 additions & 1 deletion agent/agent_test.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2677,7 +2677,6 @@ func TestAgent_ManageProcessPriority(t *testing.T) {
for _, proc := range actualProcs {
requireFileEquals(t, fs, fmt.Sprintf("/proc/%d/oom_score_adj", proc.PID), "-567")
}

})

t.Run("DisabledByDefault", func(t *testing.T) {
Expand Down