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

fix: start service when exactly the max memory is used #1322

Merged
merged 1 commit into from
Oct 21, 2023

Conversation

0utplay
Copy link
Member

@0utplay 0utplay commented Oct 21, 2023

Motivation

When having two services that require 1024mb each and a node with a memory limit of 2048mb the node would start only one of the two services.

Modification

Modified the memory check in that way that using 100% of the memory is still okay.

Result

In the example from above both services are started (how they are supposed to)

@0utplay 0utplay added v: 4.X This pull should be included in the 4.0 release in: node An issue/pull request releated to the node module code t: fix A pull request introducing a fix for a bug. labels Oct 21, 2023
@0utplay 0utplay added this to the 4.0.0-RC10 milestone Oct 21, 2023
@0utplay 0utplay requested a review from derklaro October 21, 2023 15:04
@0utplay 0utplay self-assigned this Oct 21, 2023
@0utplay 0utplay changed the title fix: start service when exactly the max memory will be used fix: start service when exactly the max memory is used Oct 21, 2023
@github-actions
Copy link

Test Results

  47 files  ±0    47 suites  ±0   1m 39s ⏱️ +20s
394 tests ±0  394 ✔️ ±0  0 💤 ±0  0 ±0 
724 runs  ±0  724 ✔️ ±0  0 💤 ±0  0 ±0 

Results for commit ae07a2e. ± Comparison against base commit 102853c.

@derklaro derklaro merged commit e852811 into nightly Oct 21, 2023
8 checks passed
@derklaro derklaro deleted the fix/max-memory-calc branch October 21, 2023 16:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
in: node An issue/pull request releated to the node module code t: fix A pull request introducing a fix for a bug. v: 4.X This pull should be included in the 4.0 release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants