Project

General

Profile

Actions
Copy link

Bug #9187

closed

Bug #8871: missing rudder_expected_reports.csv.res when starting the agent for the first time after an update

logger_rudder evaluates at each call the expected_reports_temp file

Added by Nicolas CHARLES about 8 years ago. Updated over 2 years ago.

Status:
Released
Priority:
N/A
Assignee:
Alexis Mousset
Category:
Generic methods
Target version:
ncf-0.x
Pull Request:
https://github.com/Normation/ncf/pull...
Severity:
UX impact:
User visibility:
Effort required:
Priority:
0
Name check:
Fix check:
Regression:

Description

Since the change of implementation for logger_rudder where we recreate the rudder_expected_reports.${this.promiser_pid}.csv file at each run, logger_rudder will evaluate at each call the content of the temp file due to invalid class condition:
logger_rudder_temp_resfile_repaired is always set

impact: run are slower

we ought to change this class condition in a proper way

Subtasks 1 (0 open — 1 closed)

Bug #9188: The first report is skipped after 9187ReleasedNicolas CHARLESActions
Actions
Copy link
 #1

Updated by Nicolas CHARLES about 8 years ago

  • Related to Bug #8871: missing rudder_expected_reports.csv.res when starting the agent for the first time after an update added
Actions
Copy link
 #2

Updated by Nicolas CHARLES about 8 years ago

  • Status changed from New to In progress
  • Assignee set to Nicolas CHARLES
Actions
Copy link
 #3

Updated by Nicolas CHARLES about 8 years ago

  • Status changed from In progress to Pending technical review
  • Assignee changed from Nicolas CHARLES to Alexis Mousset
  • Pull Request set to https://github.com/Normation/ncf/pull/460
Actions
Copy link
 #4

Updated by Nicolas CHARLES about 8 years ago

with 19 ncf components (12 out of the 19 being result n/a), runs are 2 seconds faster

Actions
Copy link
 #5

Updated by Nicolas CHARLES about 8 years ago

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

Applied in changeset commit:f41005db2ae6bfb29839a66b332688f39036e865.

Actions
Copy link
 #6

Updated by Vincent MEMBRÉ about 8 years ago

  • Related to deleted (Bug #8871: missing rudder_expected_reports.csv.res when starting the agent for the first time after an update)
Actions
Copy link
 #7

Updated by Vincent MEMBRÉ about 8 years ago

  • Parent task set to #8871
Actions
Copy link
 #8

Updated by Vincent MEMBRÉ about 8 years ago

  • Status changed from Pending release to Released

This bug has been fixed in ncf version that comes with Rudder 3.1.15/14 and 3.2.8/7 which were released today.

Actions
Copy link
 #9

Updated by Alexis Mousset over 2 years ago

  • Target version changed from 0.x to ncf-0.x
  • Priority set to 0
Actions
Copy link

Also available in: Atom PDF