Skip to content

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

Merged
derklaro merged 1 commit intonightlyfrom
fix/max-memory-calc
Oct 21, 2023
Merged

fix: start service when exactly the max memory is used#1322
derklaro merged 1 commit intonightlyfrom
fix/max-memory-calc

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
@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