From 5832ad3d994a97a203a21f787e18e5cb4e54f44c Mon Sep 17 00:00:00 2001 From: Nicolas Hennion Date: Wed, 3 Jan 2024 07:23:09 +0000 Subject: Docker memory usage uses the same algorithm than docker stats #2637 --- NEWS.rst | 1 + glances/outputs/static/js/components/plugin-containers.vue | 4 ++-- glances/plugins/containers/engines/docker.py | 6 ++++-- 3 files changed, 7 insertions(+), 4 deletions(-) diff --git a/NEWS.rst b/NEWS.rst index cc0c8394..26c7b355 100644 --- a/NEWS.rst +++ b/NEWS.rst @@ -23,6 +23,7 @@ Under development: https://github.com/nicolargo/glances/issues?q=is%3Aopen+is%3A alias=sda1:InternalDisk,sdb1:ExternalDisk * Alert data model change from a list of list to a list of dict #2633 + * Docker memory usage uses the same algorithm than docker stats #2637 =============== Version 3.4.0.3 diff --git a/glances/outputs/static/js/components/plugin-containers.vue b/glances/outputs/static/js/components/plugin-containers.vue index cadfc3bd..b7fb02a5 100644 --- a/glances/outputs/static/js/components/plugin-containers.vue +++ b/glances/outputs/static/js/components/plugin-containers.vue @@ -54,7 +54,7 @@ {{ $filters.number(container.cpu_percent, 1) }}
- {{ $filters.bytes(container.memory_usage_no_cache) }} + {{ $filters.bytes(container.memory_usage) }}
{{ $filters.bytes(container.limit) }} @@ -123,7 +123,7 @@ export default { 'status': containerData.status, 'uptime': containerData.uptime, 'cpu_percent': containerData.cpu.total, - 'memory_usage_no_cache': memory_usage_no_cache, + 'memory_usage': memory_usage_no_cache, 'limit': containerData.memory.limit != undefined ? containerData.memory.limit : '?', 'io_rx': containerData.io_rx != undefined ? containerData.io_rx : '?', 'io_wx': containerData.io_wx != undefined ? containerData.io_wx : '?', diff --git a/glances/plugins/containers/engines/docker.py b/glances/plugins/containers/engines/docker.py index 1c973713..bf27969c 100644 --- a/glances/plugins/containers/engines/docker.py +++ b/glances/plugins/containers/engines/docker.py @@ -120,7 +120,9 @@ class DockerStatsFetcher: def _get_memory_stats(self): """Return the container MEMORY. - Output: a dict {'usage': ..., 'limit': ..., 'max_usage': ...} + Output: a dict {'usage': ..., 'limit': ..., 'inactive_file': ...} + + Note:the displayed memory usage is 'usage - inactive_file' """ memory_stats = self._streamer.stats.get('memory_stats') @@ -131,7 +133,7 @@ class DockerStatsFetcher: stats = {field: memory_stats[field] for field in self.MANDATORY_MEMORY_FIELDS} - # Optional field + # Optional field stats:inactive_file stats['inactive_file'] = 0 if 'stats' in memory_stats: stats['inactive_file'] = memory_stats['stats'].get('inactive_file', 0) -- cgit v1.2.3