Merge pull request #916 from ryanpetrello/memcache-fact-cache-size-warning

make the fact caching plugin fail more gracefully for large payloads
This commit is contained in:
Ryan Petrello 2018-01-04 13:06:29 -05:00 committed by GitHub
commit 47875c5f9a
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23

View File

@ -92,6 +92,14 @@ class CacheModule(BaseCacheModule):
modified_key = self.translate_modified_key(key)
self.mc.set(host_key, json.dumps(value))
value = json.dumps(value)
rc = self.mc.set(host_key, value)
if rc == 0 and len(value) > self.mc.server_max_value_length:
self._display.error(
"memcache.set('{}', '?') failed, value > server_max_value_length ({} bytes)".format(
key, len(value)
)
)
self.mc.set(modified_key, datetime.datetime.now(tzutc()).isoformat())
def keys(self):