Skip to content

Commit d25c29f

Browse files
authored
Change severity of negative stats messages from WARN to DEBUG (#60375)
1 parent f8c8dcf commit d25c29f

File tree

2 files changed

+6
-6
lines changed

2 files changed

+6
-6
lines changed

server/src/main/java/org/elasticsearch/monitor/os/OsProbe.java

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -92,7 +92,7 @@ public long getFreePhysicalMemorySize() {
9292
try {
9393
final long freeMem = (long) getFreePhysicalMemorySize.invoke(osMxBean);
9494
if (freeMem < 0) {
95-
logger.warn("OS reported a negative free memory value [{}]", freeMem);
95+
logger.debug("OS reported a negative free memory value [{}]", freeMem);
9696
return 0;
9797
}
9898
return freeMem;
@@ -113,7 +113,7 @@ public long getTotalPhysicalMemorySize() {
113113
try {
114114
final long totalMem = (long) getTotalPhysicalMemorySize.invoke(osMxBean);
115115
if (totalMem < 0) {
116-
logger.warn("OS reported a negative total memory value [{}]", totalMem);
116+
logger.debug("OS reported a negative total memory value [{}]", totalMem);
117117
return 0;
118118
}
119119
return totalMem;
@@ -134,7 +134,7 @@ public long getFreeSwapSpaceSize() {
134134
try {
135135
final long mem = (long) getFreeSwapSpaceSize.invoke(osMxBean);
136136
if (mem < 0) {
137-
logger.warn("OS reported a negative free swap space size [{}]", mem);
137+
logger.debug("OS reported a negative free swap space size [{}]", mem);
138138
return 0;
139139
}
140140
return mem;
@@ -155,7 +155,7 @@ public long getTotalSwapSpaceSize() {
155155
try {
156156
final long mem = (long) getTotalSwapSpaceSize.invoke(osMxBean);
157157
if (mem < 0) {
158-
logger.warn("OS reported a negative total swap space size [{}]", mem);
158+
logger.debug("OS reported a negative total swap space size [{}]", mem);
159159
return 0;
160160
}
161161
return mem;

server/src/main/java/org/elasticsearch/monitor/os/OsStats.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -221,7 +221,7 @@ public ByteSizeValue getUsed() {
221221
// We intentionally check for (total == 0) rather than (total - free < 0) so as not to hide
222222
// cases where (free > total) which would be a different bug.
223223
if (free > 0) {
224-
logger.warn("cannot compute used swap when total swap is 0 and free swap is " + free);
224+
logger.debug("cannot compute used swap when total swap is 0 and free swap is " + free);
225225
}
226226
return new ByteSizeValue(0);
227227
}
@@ -283,7 +283,7 @@ public ByteSizeValue getUsed() {
283283
// We intentionally check for (total == 0) rather than (total - free < 0) so as not to hide
284284
// cases where (free > total) which would be a different bug.
285285
if (free > 0) {
286-
logger.warn("cannot compute used memory when total memory is 0 and free memory is " + free);
286+
logger.debug("cannot compute used memory when total memory is 0 and free memory is " + free);
287287
}
288288
return new ByteSizeValue(0);
289289
}

0 commit comments

Comments
 (0)