Project

General

Profile

Actions

Bug #5884

closed

Architecture #5305: Rewrite techniques so they all use ncf report methods

Architecture #5353: Rewrite reports to use rudder_common_report

Architecture #5689: Rewrite system techniques to use rudder_common_report

Duplicate report introduced in endExecution bundle

Added by Jonathan CLARKE over 9 years ago. Updated about 9 years ago.

Status:
Released
Priority:
2
Category:
Techniques
Target version:
Severity:
UX impact:
User visibility:
Effort required:
Priority:
Name check:
Fix check:
Regression:

Description

As seen in commit:52d4fd20606425cf0403f6b3cea50fe0a5b05296, a duplicate reports line was introduced into promises.st in the endExecution bundle:

   reports:
     start_run_message_sent::
       "@@Common@@log_info@@&TRACKINGKEY&@@common@@EndRun@@${g.execRun}##${g.uuid}@#End execution with config [&RUDDER_NODE_CONFIG_ID&]";

+  reports:
+    cfengine_3::
+      "@@Common@@log_info@@&TRACKINGKEY&@@common@@EndRun@@${g.execRun}##${g.uuid}@#End execution with config [&RUDDER_NODE_CONFIG_ID&]";

This causes trouble in changes-only compliance mode, because the later report is always output. It does, however, no harm in full-compliance mode, since CFEngine considers these 2 promises identical, and only executes it once.

We need to remove the dupe.

Actions #1

Updated by Jonathan CLARKE over 9 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/rudder-techniques/pull/586
Actions #2

Updated by Jonathan CLARKE over 9 years ago

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

Applied in changeset commit:24e8416717ba32a1b67d2edc35d6cdf39694594e.

Actions #3

Updated by Vincent MEMBRÉ over 9 years ago

  • Status changed from Pending release to Released

This bug has been fixed in Rudder 3.0.0~beta1 which was release on 01/12/2014.

Actions #4

Updated by Benoît PECCATTE about 9 years ago

  • Category set to Techniques
Actions

Also available in: Atom PDF