Project

General

Profile

Actions
Copy link

Bug #6149

closed

Bug #5389: When two techniques generated with ncf-builder use the same generic method with same parameters, reporting cannot dinstinguish them, and reports "Unknown"

Update dependency between Rudder and ncf

Bug #6149: Update dependency between Rudder and ncf

Added by Nicolas CHARLES over 11 years ago. Updated over 11 years ago.

Status:
Released
Priority:
1 (highest)
Assignee:
Benoît PECCATTE
Category:
-
Target version:
2.10.10
Pull Request:
https://github.com/Normation/rudder-p...
Severity:
UX impact:
User visibility:
Effort required:
Priority:
Name check:
Fix check:
Regression:

Description

Ticket #5389 and #4806 include changes in the logger_rudder reports, so we need to force the upgrade of ncf when user will upgrade rudder.

the mandatory version will be the one just after this merge
https://github.com/Normation/ncf/pull/167

Related issues 3 (0 open — 3 closed)

Related to Rudder - Bug #4806: When two techniques generated with ncf-builder use the same generic method with same parameters, reporting cannot dinstinguish them, and reports "Unknown"ReleasedBenoît PECCATTEActions
Related to Rudder - Bug #6168: ncf API hooks fails because it uses bashism but declares using shellReleasedBenoît PECCATTEActions
Related to Rudder - Bug #6170: Rudder reports are made on ncf only if class "full_compliance" is set, but this class is defined only in Rudder 3.0ReleasedBenoît PECCATTEActions

Updated by Nicolas CHARLES over 11 years ago Actions
Copy link
 #1

  • Priority changed from N/A to 1 (highest)

Updated by Matthieu CERDA over 11 years ago Actions
Copy link
 #2

  • Status changed from New to In progress

Updated by Matthieu CERDA over 11 years ago Actions
Copy link
 #3

Since a new lil' bug has been found in the meantime ( #6168 ) , and merged today at 14:46, I'll update the dependency to today, 15:00

Updated by Matthieu CERDA over 11 years ago Actions
Copy link
 #4

  • Status changed from In progress to Pending technical review
  • Assignee changed from Matthieu CERDA to Benoît PECCATTE
  • % Done changed from 0 to 100
  • Pull Request set to https://github.com/Normation/rudder-packages/pull/595

Updated by Matthieu CERDA over 11 years ago Actions
Copy link
 #5

  • Pull Request changed from https://github.com/Normation/rudder-packages/pull/595 to https://github.com/Normation/rudder-packages/pull/597

PR updated.

Updated by Nicolas CHARLES over 11 years ago Actions
Copy link
 #6

Ok, after digging down into the bugs, after the merge of https://github.com/Normation/ncf/pull/168 should be ok !

Updated by Matthieu CERDA over 11 years ago Actions
Copy link
 #7

Also, we need #6170 ! Updating to today, 16h40.

Updated by Matthieu CERDA over 11 years ago Actions
Copy link
 #8

  • Status changed from Pending technical review to Pending release

Applied in changeset packages:rudder-packages|commit:e6f80e2ce8f447731156e9e00bc9810b69539086.

Updated by Benoît PECCATTE over 11 years ago Actions
Copy link
 #9

Applied in changeset packages:rudder-packages|commit:a629ccb745ec6619f984946efa984cf172f9e84c.

Updated by Vincent MEMBRÉ over 11 years ago Actions
Copy link
 #10

  • Target version changed from 2.10.9 to 2.10.10

Updated by Vincent MEMBRÉ over 11 years ago Actions
Copy link
 #11

  • Parent task set to #5389

Updated by Vincent MEMBRÉ over 11 years ago Actions
Copy link
 #12

  • Status changed from Pending release to Released

This bug has been fixed in Rudder 2.10.10 and 2.11.7, which were these days.

Actions
Copy link

Also available in: PDF Atom