Browse Source

Change severity of negative stats messages from WARN to DEBUG (#60375)

Dan Hermann 5 years ago
parent
commit
d25c29fb4d

+ 4 - 4
server/src/main/java/org/elasticsearch/monitor/os/OsProbe.java

@@ -92,7 +92,7 @@ public class OsProbe {
         try {
             final long freeMem = (long) getFreePhysicalMemorySize.invoke(osMxBean);
             if (freeMem < 0) {
-                logger.warn("OS reported a negative free memory value [{}]", freeMem);
+                logger.debug("OS reported a negative free memory value [{}]", freeMem);
                 return 0;
             }
             return freeMem;
@@ -113,7 +113,7 @@ public class OsProbe {
         try {
             final long totalMem = (long) getTotalPhysicalMemorySize.invoke(osMxBean);
             if (totalMem < 0) {
-                logger.warn("OS reported a negative total memory value [{}]", totalMem);
+                logger.debug("OS reported a negative total memory value [{}]", totalMem);
                 return 0;
             }
             return totalMem;
@@ -134,7 +134,7 @@ public class OsProbe {
         try {
             final long mem = (long) getFreeSwapSpaceSize.invoke(osMxBean);
             if (mem < 0) {
-                logger.warn("OS reported a negative free swap space size [{}]", mem);
+                logger.debug("OS reported a negative free swap space size [{}]", mem);
                 return 0;
             }
             return mem;
@@ -155,7 +155,7 @@ public class OsProbe {
         try {
             final long mem = (long) getTotalSwapSpaceSize.invoke(osMxBean);
             if (mem < 0) {
-                logger.warn("OS reported a negative total swap space size [{}]", mem);
+                logger.debug("OS reported a negative total swap space size [{}]", mem);
                 return 0;
             }
             return mem;

+ 2 - 2
server/src/main/java/org/elasticsearch/monitor/os/OsStats.java

@@ -221,7 +221,7 @@ public class OsStats implements Writeable, ToXContentFragment {
                 // We intentionally check for (total == 0) rather than (total - free < 0) so as not to hide
                 // cases where (free > total) which would be a different bug.
                 if (free > 0) {
-                    logger.warn("cannot compute used swap when total swap is 0 and free swap is " + free);
+                    logger.debug("cannot compute used swap when total swap is 0 and free swap is " + free);
                 }
                 return new ByteSizeValue(0);
             }
@@ -283,7 +283,7 @@ public class OsStats implements Writeable, ToXContentFragment {
                 // We intentionally check for (total == 0) rather than (total - free < 0) so as not to hide
                 // cases where (free > total) which would be a different bug.
                 if (free > 0) {
-                    logger.warn("cannot compute used memory when total memory is 0 and free memory is " + free);
+                    logger.debug("cannot compute used memory when total memory is 0 and free memory is " + free);
                 }
                 return new ByteSizeValue(0);
             }