Project

General

Profile

Bug #8430

Bug #8343: logger_rudder breaks if expected_reports.csv.res is missing but expected_reports.csv.tmp is present

logger_rudder can re-read expected_reports_temp even without change, impacting performance significantly

Added by Jonathan CLARKE over 4 years ago. Updated over 4 years ago.

Status:
Released
Priority:
N/A
Assignee:
Nicolas CHARLES
Category:
-
Target version:
0.x
Pull Request:
https://github.com/Normation/ncf/pull...
Severity:
User visibility:
Effort required:
Priority:

Description

On one test machine, this took an agent run from 53 seconds to 124.

Somehow, the local classes in bundle logger_rudder can "persist" between calls - I think this is when the bundle is called several times in a single "call". This can be simply worked around using an extra class protection.

Subtasks

Bug #8433: logger_rudder evaluates too late content of temp file, causing it to miss first reportReleasedBenoît PECCATTE2016-06-01Actions

Related issues

Related to ncf - Bug #8379: invalid reporting in ncfReleasedNicolas CHARLES2016-05-25Actions
#1

Updated by Jonathan CLARKE over 4 years ago

  • Status changed from In progress to Pending technical review
  • Assignee changed from Jonathan CLARKE to Nicolas CHARLES
  • Pull Request set to https://github.com/Normation/ncf/pull/371
#2

Updated by Jonathan CLARKE over 4 years ago

  • Status changed from Pending technical review to Pending release
  • % Done changed from 0 to 100
#3

Updated by Nicolas CHARLES over 4 years ago

  • Related to Bug #8379: invalid reporting in ncf added
#4

Updated by Vincent MEMBRÉ over 4 years ago

  • Status changed from Pending release to Released

Also available in: Atom PDF