RHINENG-14707: Reduce trace data for Patch issues (#575)

RedHatInsights · Dec 4, 2024 · 067c8d8 · 067c8d8
1 parent 26cbff5
commit 067c8d8
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 5 deletions.
diff --git a/src/generator/factories/CVEFactory.js b/src/generator/factories/CVEFactory.js
@@ -20,14 +20,14 @@ module.exports = class CVEFactory extends Factory {
             throw errors.unknownIssue(id);
         }
 
-        trace.event(`Disambiguate resolution: ${JSON.stringify(resolutions)}`);
+        trace.event(`Disambiguate resolution...`);
         const disambiguatedResolution = this.disambiguate(resolutions, resolution, id, strict);
-        trace.event(`Disambiguated resolution: ${disambiguatedResolution}`);
+        trace.event(`Disambiguated resolution.`);
 
         trace.event('Create erratum play...');
         const result = new ErratumPlay(id, hosts, disambiguatedResolution, disambiguatedResolution.description);
 
-        trace.leave(`Returning: ${JSON.stringify(result)}`);
+        trace.leave();
         return result;
     }
 };
diff --git a/src/generator/generator.controller.js b/src/generator/generator.controller.js
@@ -75,7 +75,6 @@ exports.playbookPipeline = async function ({issues, auto_reboot = true}, remedia
             version: resolution.version || null,
             hosts}))
     };
-    trace.event(`Canonical definition: ${JSON.stringify(definition)}`);
 
     trace.event('Aggregate erratum plays...');
     issues = erratumPlayAggregator.process(issues);
@@ -128,7 +127,9 @@ exports.resolveSystems = async function (issues, strict = true) {
     trace.enter('generator.controller.resolveSystems');
 
     const systemIds = _(issues).flatMap('systems').uniq().value();
-    trace.event(`System IDs: ${JSON.stringify(systemIds)}`);
+    if (systemIds.length <= 25) { // avoid logging huge list...
+        trace.event(`System IDs: ${JSON.stringify(systemIds)}`);
+    }
 
     // bypass cache as ansible_host may change so we want to grab the latest one
     trace.event('Get system details...');