check-redis-memory-percentage: Use total memory if max is 0 (sensu-pl…

…ugins#29)

* check-redis-memory-percentage: Use total memory if max is 0

* check-redis-memory-percentage: Changing memory casting and using fdiv

CHANGELOG.md

@@ -4,6 +4,9 @@ This project adheres to [Semantic Versioning](http://semver.org/).
 This CHANGELOG follows the format listed at [Keep A Changelog](http://keepachangelog.com/)
 
 ## [Unreleased]
+### Changed
+- check-redis-memory-percentage: Handle case where maxmemory is 0 (@stevenviola)
+
 ## [1.1.1] - 2017-05-05
 ### Fixed
 - updated gemspec to avoid deprecation warnings in redis client for Fixnum

bin/check-redis-memory-percentage.rb

@@ -80,8 +80,12 @@ def run
     redis = Redis.new(options)
 
     redis_info = redis.info
-    memory_in_use = redis_info.fetch('used_memory').to_f.div(1024).to_f             # used memory in KB (KiloBytes)
-    total_memory = redis_info.fetch('maxmemory', system_memory).to_f.div(1024).to_f # max memory (if specified) in KB
+    max_memory = redis_info.fetch('maxmemory', 0).to_i
+    if max_memory == 0
+      max_memory = redis_info.fetch('total_system_memory', system_memory).to_i
+    end
+    memory_in_use = redis_info.fetch('used_memory').to_i.fdiv(1024) # used memory in KB (KiloBytes)
+    total_memory = max_memory.fdiv(1024)                            # max memory (if specified) in KB
 
     used_memory = ((memory_in_use / total_memory) * 100).round(2)
     warn_memory = config[:warn_mem]