From 0b6b871144645494cc3fe9fb10785ff524353db3 Mon Sep 17 00:00:00 2001 From: KernelDeimos <7225168+KernelDeimos@users.noreply.github.com> Date: Fri, 10 Oct 2025 16:55:31 -0400 Subject: [PATCH] devex: disable logs for missing fsentries and cache Often fsentry stats are used to see if an fsentry exists, so fsentry not found logs are common and expected. This commit disables those by default. Additionally, logging for cache events is disabled as well, since those are only helpful when debugging cache specifically. --- .../src/modules/puterfs/lib/PuterFSProvider.js | 11 ++++++++++- .../src/services/file-cache/FileCacheService.js | 5 ++++- 2 files changed, 14 insertions(+), 2 deletions(-) diff --git a/src/backend/src/modules/puterfs/lib/PuterFSProvider.js b/src/backend/src/modules/puterfs/lib/PuterFSProvider.js index ea69da69..c23b8abb 100644 --- a/src/backend/src/modules/puterfs/lib/PuterFSProvider.js +++ b/src/backend/src/modules/puterfs/lib/PuterFSProvider.js @@ -47,6 +47,13 @@ class PuterFSProvider extends putility.AdvancedBase { _path: require('path'), uuidv4: require('uuid').v4, config: require('../../../config.js'), + }; + + constructor (...a) { + super(...a); + + this.log_fsentriesNotFound = (this.modules.config.logging ?? []) + .includes('fsentries-not-found'); } get_capabilities () { @@ -185,7 +192,9 @@ class PuterFSProvider extends putility.AdvancedBase { } if ( ! entry ) { - controls.log.warn(`entry not found: ${selector.describe(true)}`); + if ( this.log_fsentriesNotFound ) { + controls.log.warn(`entry not found: ${selector.describe(true)}`); + } } if ( entry === null || typeof entry !== 'object' ) { diff --git a/src/backend/src/services/file-cache/FileCacheService.js b/src/backend/src/services/file-cache/FileCacheService.js index e4aaea72..4f8c4dae 100644 --- a/src/backend/src/services/file-cache/FileCacheService.js +++ b/src/backend/src/services/file-cache/FileCacheService.js @@ -53,7 +53,7 @@ class FileCacheService extends AdvancedBase { path_: require('path'), } - constructor ({ services, my_config }) { + constructor ({ services, my_config, config: global_config }) { super({ services }); this.log = services.get('log-service').create(this.constructor.name); @@ -74,6 +74,9 @@ class FileCacheService extends AdvancedBase { initial: 0.5, alpha: 0.2, }); + + this.logging_enabled = (global_config.logging ?? []) + .includes('file-cache'); this.init();