Skip to content
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

[WIP] Set container oom killed state correctly #17756

Closed
wants to merge 1 commit into from

Conversation

umohnani8
Copy link
Member

Parse the oom file created by conmon when a container is oomkilled and set the oomkilled value to true in the container state. conmon writes the oom file in a persist directory that podman checks now for any oom files for a given container.

Partially fixes #13102

Signed-off-by: Urvashi Mohnani [email protected]

Does this PR introduce a user-facing change?

Set container oomkilled state correctly if it was oomkilled.

@openshift-ci openshift-ci bot added release-note do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. labels Mar 13, 2023
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Mar 13, 2023

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: umohnani8

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-ci openshift-ci bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Mar 13, 2023
@umohnani8 umohnani8 force-pushed the memory branch 6 times, most recently from f7dac91 to 73b906b Compare April 7, 2023 18:49
@rhatdan
Copy link
Member

rhatdan commented Apr 10, 2023

@umohnani8 Any progress on this one?

@umohnani8
Copy link
Member Author

@rhatdan yeah I am on it, I have been trying to figure out and reproduce the test failures.

@github-actions
Copy link

A friendly reminder that this PR had no activity for 30 days.

Parse the oom file created by conmon when a container is
oomkilled and set the oomkilled value to true in the container
state. conmon writes the oom file in a persist directory that
podman checks now for any oom files for a given container.

Signed-off-by: Urvashi Mohnani <[email protected]>
@umohnani8
Copy link
Member Author

Replaced by #21523

@umohnani8 umohnani8 closed this Feb 7, 2024
@stale-locking-app stale-locking-app bot added the locked - please file new issue/PR Assist humans wanting to comment on an old issue or PR with locked comments. label May 8, 2024
@stale-locking-app stale-locking-app bot locked as resolved and limited conversation to collaborators May 8, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. locked - please file new issue/PR Assist humans wanting to comment on an old issue or PR with locked comments. release-note
Projects
None yet
2 participants