Skip to content

OCPBUGS-58324: podman-etcd Add OOM score adjustment for etcd containers #2060

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.

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

Conversation

clobrano
Copy link
Collaborator

@clobrano clobrano commented Jul 3, 2025

This change introduces a new oom parameter to the podman-etcd OCF agent. This allows tuning the Out-Of-Memory (OOM) score adjustment for the etcd container.

The oom parameter accepts integer values from -1000 to 1000, defaulting to -997 (system-node-critical equivalent).

see https://kubernetes.io/docs/concepts/scheduling-eviction/node-pressure-eviction/#node-out-of-memory-behavior

Key changes:

  • Added OCF_RESKEY_oom parameter to agent definition (content type="integer").
  • Integrated --oom-score-adj option into podman_start().
  • Implemented input validation for oom in podman_validate(), ensuring values are within the [-1000:1000] range.

@clobrano clobrano marked this pull request as draft July 3, 2025 14:06
Copy link

knet-jenkins bot commented Jul 3, 2025

Can one of the admins check and authorise this run please: https://ci.kronosnet.org/job/resource-agents/job/resource-agents-pipeline/job/PR-2060/1/input

This change introduces a new `oom` parameter to the `podman-etcd` OCF
agent. This allows tuning the Out-Of-Memory (OOM) score adjustment for
the etcd container.

The `oom` parameter accepts integer values from -1000 to 1000,
defaulting to -997 (system-node-critical equivalent).

see https://kubernetes.io/docs/concepts/scheduling-eviction/node-pressure-eviction/#node-out-of-memory-behavior

Key changes:
- Added `OCF_RESKEY_oom` parameter to agent definition (`content type="integer"`).
- Integrated `--oom-score-adj` option into `podman_start()`.
- Implemented input validation for `oom` in `podman_validate()`,
  ensuring values are within the [-1000:1000] range.
@clobrano clobrano force-pushed the OCPBUGS-58324-podman-etcd-oom-score-adj branch from 149f13a to d08a7f7 Compare July 3, 2025 15:23
Copy link

knet-jenkins bot commented Jul 3, 2025

Can one of the admins check and authorise this run please: https://ci.kronosnet.org/job/resource-agents/job/resource-agents-pipeline/job/PR-2060/2/input

@clobrano clobrano changed the title [WIP] OCPBUGS-58324: podman-etcd Add OOM score adjustment for etcd containers OCPBUGS-58324: podman-etcd Add OOM score adjustment for etcd containers Jul 7, 2025
@clobrano clobrano marked this pull request as ready for review July 7, 2025 09:20
@oalbrigt
Copy link
Contributor

oalbrigt commented Jul 9, 2025

retest this please

@oalbrigt oalbrigt merged commit 968ad70 into ClusterLabs:main Jul 9, 2025
1 check failed
@oalbrigt
Copy link
Contributor

oalbrigt commented Jul 9, 2025

Thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants