Skip to content

remove memory detect code#88

Merged
hiverge-robot merged 1 commit intohiverge:mainfrom
kerthcet:cleanup/remove-memory
Jan 9, 2026
Merged

remove memory detect code#88
hiverge-robot merged 1 commit intohiverge:mainfrom
kerthcet:cleanup/remove-memory

Conversation

@kerthcet
Copy link
Collaborator

@kerthcet kerthcet commented Jan 9, 2026

We're using Pod container which already limits the memory usage by container runtime like containerd, no need to detect ourselves.

Signed-off-by: kerthcet <kerthcet@gmail.com>
@hiverge-robot hiverge-robot added needs-triage Indicates an issue or PR lacks a label and requires one. needs-priority Indicates a PR lacks a label and requires one. do-not-merge/needs-kind Indicates a PR lacks a label and requires one. approved Indicates a PR has been approved by an approver from all required OWNERS files. labels Jan 9, 2026
@kerthcet
Copy link
Collaborator Author

kerthcet commented Jan 9, 2026

/kind cleanup

@hiverge-robot hiverge-robot added cleanup Categorizes issue or PR as related to cleaning up code, process, or technical debt. and removed do-not-merge/needs-kind Indicates a PR lacks a label and requires one. labels Jan 9, 2026
@kerthcet
Copy link
Collaborator Author

kerthcet commented Jan 9, 2026

cc @brp-hiverge

@brp-hiverge
Copy link
Contributor

/lgtm
/approve

@hiverge-robot hiverge-robot added the lgtm Looks good to me, indicates that a PR is ready to be merged. label Jan 9, 2026
@hiverge-robot hiverge-robot merged commit 5209a46 into hiverge:main Jan 9, 2026
41 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. cleanup Categorizes issue or PR as related to cleaning up code, process, or technical debt. lgtm Looks good to me, indicates that a PR is ready to be merged. needs-priority Indicates a PR lacks a label and requires one. needs-triage Indicates an issue or PR lacks a label and requires one.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants