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

Added by Nicolas CHARLES almost 10 years ago. Updated almost 10 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 PECCATTE2015-01-22Actions
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
Actions
Copy link
 #1

Updated by Nicolas CHARLES almost 10 years ago

  • Priority changed from N/A to 1 (highest)
Actions
Copy link
 #2

Updated by Matthieu CERDA almost 10 years ago

  • Status changed from New to In progress
Actions
Copy link
 #3

Updated by Matthieu CERDA almost 10 years ago

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

Actions
Copy link
 #4

Updated by Matthieu CERDA almost 10 years ago

  • 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
Actions
Copy link
 #5

Updated by Matthieu CERDA almost 10 years ago

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

PR updated.

Actions
Copy link
 #6

Updated by Nicolas CHARLES almost 10 years ago

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

Actions
Copy link
 #7

Updated by Matthieu CERDA almost 10 years ago

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

Actions
Copy link
 #8

Updated by Matthieu CERDA almost 10 years ago

  • Status changed from Pending technical review to Pending release

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

Actions
Copy link
 #9

Updated by Benoît PECCATTE almost 10 years ago

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

Actions
Copy link
 #10

Updated by Vincent MEMBRÉ almost 10 years ago

  • Target version changed from 2.10.9 to 2.10.10
Actions
Copy link
 #11

Updated by Vincent MEMBRÉ almost 10 years ago

  • Parent task set to #5389
Actions
Copy link
 #12

Updated by Vincent MEMBRÉ almost 10 years ago

  • 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: Atom PDF