Merge pull request #1798 from tgstation/BadWarning

Fix a bad warning
tgstation · Mar 4, 2024 · 8332ae8 · 8332ae8
2 parents 0152f06 + 4a5610c
commit 8332ae8
Showing 1 changed file with 5 additions and 4 deletions.
diff --git a/src/Tgstation.Server.Host/System/PosixProcessFeatures.cs b/src/Tgstation.Server.Host/System/PosixProcessFeatures.cs
@@ -172,10 +172,11 @@ public async Task StartAsync(CancellationToken cancellationToken)
 			var originalOomAdjust = Int16.Parse(trimmedString, CultureInfo.InvariantCulture);
 			baselineOomAdjust = Math.Clamp(originalOomAdjust, (short)-1000, (short)1000);
 
-			if (originalOomAdjust != baselineOomAdjust)
-				logger.LogWarning("oom_score_adj is at it's limit of 1000 (Clamped from {original}). TGS cannot guarantee the kill order of its parent/child processes!", originalOomAdjust);
-			else
-				logger.LogWarning("oom_score_adj is at it's limit of 1000. TGS cannot guarantee the kill order of its parent/child processes!");
+			if (baselineOomAdjust == 1000)
+				if (originalOomAdjust != baselineOomAdjust)
+					logger.LogWarning("oom_score_adj is at it's limit of 1000 (Clamped from {original}). TGS cannot guarantee the kill order of its parent/child processes!", originalOomAdjust);
+				else
+					logger.LogWarning("oom_score_adj is at it's limit of 1000. TGS cannot guarantee the kill order of its parent/child processes!");
 
 			try
 			{