Bug #6343: When a technique reports several time the exact same text, there is only one report generated - Rudder - Issue Tracker

Custom queries

8.1 beta
Bugs triage
CS - by priority
CS - to review
My TR
Need technical review
Open bugs (all)
Open bugs (UI - UX)
Open UX defects (all)
Opened by users

Actions

Copy link

Bug #6343

closed

When a technique reports several time the exact same text, there is only one report generated

Added by Nicolas CHARLES over 9 years ago. Updated over 6 years ago.

Status:

Released

Priority:

N/A

Assignee:

Nicolas CHARLES

Category:

System techniques

Target version:

4.1.13

Pull Request:

https://github.com/Normation/rudder-t...

Severity:

Major - prevents use of part of Rudder | no simple workaround

UX impact:

User visibility:

Getting started - demo | first install | level 1 Techniques

Effort required:

Priority:

Name check:

Fix check:

Regression:

Description

In the Technique refactoring in Rudder 3.0 to use rudder_common_reports, we introduced a bug which prevent to have duplicate identical reports.

The main problem is that we are calling a bundle to do reports, with exactly the same parameters, and in this bundle, the agent detects that he already handled this promises, and skip it !

This bug appears when in a given directive, we are managing twice (or more) the same object.

A workaround would be to create a new version of rudder_common_report and rudder_common_report_generic, with a new parameter (the current index), and se this index as an handle or comment of the usebundle and reports.
This permit the agent to distinguish each promises.

This should go at least in 3.0, but could go to 2.11 to allow backporting of techniques

Subtasks 7 (0 open — 7 closed)

Bug #6311: Missing reports when editing twice the same file with enforce a file content v7.0	Released	Nicolas CHARLES	2015-03-02	Actions
Bug #6465: Update rudder_common_report and rudder_common_reports_generic to be compatible wiht multi instance technique	Released	Benoît PECCATTE	2015-04-07	Actions
Bug #6469: Typo in rudder_common_reports_generic	Released	Benoît PECCATTE	2015-04-07	Actions
Bug #12882: same as parent ticket for branch 4.3	Released	Alexis Mousset		Actions
Bug #12941: typo introduced by parent ticket -branch 4.3)	Released	Alexis Mousset		Actions
Bug #12889: spurious rudder_common_report_indexs_generic_index introduced by parent ticket	Released	Alexis Mousset		Actions
Bug #12917: typo introduced by parent ticket	Released	Alexis Mousset		Actions

History
Notes
Property changes
Associated revisions

Actions

Copy link

Updated by Nicolas CHARLES over 9 years ago

Status changed from New to Discussion

Actions

Copy link

Updated by Vincent MEMBRÉ over 9 years ago

Target version changed from 2.11.8 to 2.11.9

Actions

Copy link

Updated by Benoît PECCATTE over 9 years ago

We should change Rudder internal API in the following way:

- add a new report bundle that take a new parameter, named or call_id or ...
- call this bundle instead of rudder_common_report in every technique that is multi-instance or can have multivalued components
- use the existing index for multivaluation in the technique as the new parameter
- On old Rudder versions the new bundle should create a report with message containing the index (as nicely as possible)
- On new Rudder versions the new bundle should create a report with a new field containing the index that is parsable by Rudder

Actions

Copy link

Updated by Benoît PECCATTE over 9 years ago

Status changed from Discussion to 8
Assignee deleted (~~Benoît PECCATTE~~)

Actions

Copy link

Updated by Vincent MEMBRÉ over 9 years ago

Target version changed from 2.11.9 to 2.11.10

Actions

Copy link

Updated by Nicolas CHARLES over 9 years ago

I checked, and this fix should go in 2.10 as well, there are multininstance technique there using rudder_common_report

Actions

Copy link

Updated by Benoît PECCATTE over 9 years ago

Status changed from 8 to New

Actions

Copy link

Updated by Vincent MEMBRÉ over 9 years ago

Target version changed from 2.11.10 to 2.11.11

Actions

Copy link

Updated by Vincent MEMBRÉ over 9 years ago

Target version changed from 2.11.11 to 2.11.12

Actions

Copy link

#10

Updated by Vincent MEMBRÉ over 9 years ago

Target version changed from 2.11.12 to 2.11.13

Actions

Copy link

#11

Updated by Vincent MEMBRÉ over 9 years ago

Target version changed from 2.11.13 to 2.11.14

Actions

Copy link

#12

Updated by Vincent MEMBRÉ about 9 years ago

Target version changed from 2.11.14 to 2.11.15

Actions

Copy link

#13

Updated by Vincent MEMBRÉ about 9 years ago

Target version changed from 2.11.15 to 2.11.16

Actions

Copy link

#14

Updated by Vincent MEMBRÉ about 9 years ago

Target version changed from 2.11.16 to 2.11.17

Actions

Copy link

#15

Updated by Vincent MEMBRÉ almost 9 years ago

Target version changed from 2.11.17 to 2.11.18

Actions

Copy link

#16

Updated by Vincent MEMBRÉ almost 9 years ago

Target version changed from 2.11.18 to 2.11.19

Actions

Copy link

#17

Updated by Vincent MEMBRÉ over 8 years ago

Target version changed from 2.11.19 to 2.11.20

Actions

Copy link

#18

Updated by Vincent MEMBRÉ over 8 years ago

Target version changed from 2.11.20 to 2.11.21

Actions

Copy link

#19

Updated by Vincent MEMBRÉ over 8 years ago

Target version changed from 2.11.21 to 2.11.22

Actions

Copy link

#20

Updated by Vincent MEMBRÉ over 8 years ago

Target version changed from 2.11.22 to 2.11.23

Actions

Copy link

#21

Updated by Vincent MEMBRÉ over 8 years ago

Target version changed from 2.11.23 to 2.11.24

Actions

Copy link

#22

Updated by Vincent MEMBRÉ about 8 years ago

Target version changed from 2.11.24 to 308

Actions

Copy link

#23

Updated by Vincent MEMBRÉ about 8 years ago

Target version changed from 308 to 3.1.14

Actions

Copy link

#24

Updated by Vincent MEMBRÉ about 8 years ago

Target version changed from 3.1.14 to 3.1.15

Actions

Copy link

#25

Updated by Vincent MEMBRÉ about 8 years ago

Target version changed from 3.1.15 to 3.1.16

Actions

Copy link

#26

Updated by Vincent MEMBRÉ about 8 years ago

Target version changed from 3.1.16 to 3.1.17

Actions

Copy link

#27

Updated by Vincent MEMBRÉ almost 8 years ago

Target version changed from 3.1.17 to 3.1.18

Actions

Copy link

#28

Updated by Vincent MEMBRÉ almost 8 years ago

Target version changed from 3.1.18 to 3.1.19

Actions

Copy link

#29

Updated by Jonathan CLARKE over 7 years ago

Severity set to Major - prevents use of part of Rudder | no simple workaround
User visibility set to Getting started - demo | first install | level 1 Techniques

Actions

Copy link

#30

Updated by Benoît PECCATTE over 7 years ago

Priority set to 45

Actions

Copy link

#31

Updated by Vincent MEMBRÉ over 7 years ago

Target version changed from 3.1.19 to 3.1.20

Actions

Copy link

#32

Updated by Vincent MEMBRÉ over 7 years ago

Target version changed from 3.1.20 to 3.1.21

Actions

Copy link

#33

Updated by Vincent MEMBRÉ over 7 years ago

Target version changed from 3.1.21 to 3.1.22

Actions

Copy link

#34

Updated by Benoît PECCATTE over 7 years ago

Priority changed from 45 to 58

Actions

Copy link

#35

Updated by Vincent MEMBRÉ over 7 years ago

Target version changed from 3.1.22 to 3.1.23

Actions

Copy link

#36

Updated by Alexis Mousset over 7 years ago

Subject changed from When a Rudder 3.0 Technique reports several time the same the same text, there is only one report generated to When a technique reports several time the exact same text, there is only one report generated

Actions

Copy link

#37

Updated by Alexis Mousset over 7 years ago

rudder_common_report_index allows avoiding this issue. I'm not closing this issue as it is likely not used everywhere.

Actions

Copy link

#38

Updated by Vincent MEMBRÉ about 7 years ago

Target version changed from 3.1.23 to 3.1.24

Actions

Copy link

#39

Updated by Vincent MEMBRÉ about 7 years ago

Target version changed from 3.1.24 to 3.1.25

Actions

Copy link

#40

Updated by Vincent MEMBRÉ almost 7 years ago

Target version changed from 3.1.25 to 387

Actions

Copy link

#41

Updated by Vincent MEMBRÉ almost 7 years ago

Target version changed from 387 to 4.1.10

Actions

Copy link

#42

Updated by Vincent MEMBRÉ almost 7 years ago

Target version changed from 4.1.10 to 4.1.11

Actions

Copy link

#43

Updated by Vincent MEMBRÉ over 6 years ago

Target version changed from 4.1.11 to 4.1.12
Priority changed from 58 to 59

Actions

Copy link

#44

Updated by Vincent MEMBRÉ over 6 years ago

Target version changed from 4.1.12 to 4.1.13

Actions

Copy link

#45

Updated by Vincent MEMBRÉ over 6 years ago

Assignee set to Nicolas CHARLES
Priority changed from 59 to 60

Actions

Copy link

#46

Updated by Nicolas CHARLES over 6 years ago

Status changed from New to In progress

Actions

Copy link

#47

Updated by Nicolas CHARLES over 6 years ago

i'm using this expression to replace

sed -i "s/\(.*\)rudder_common_report\(.*\)\"),/\1rudder_common_report_index\2\", \"\${index}\"),/"

Actions

Copy link

#48

Updated by Nicolas CHARLES over 6 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/rudder-techniques/pull/1321

PR https://github.com/Normation/rudder-techniques/pull/1321

Actions

Copy link

#49

Updated by Rudder Quality Assistant over 6 years ago

Assignee changed from Alexis Mousset to Nicolas CHARLES

Actions

Copy link

#50

Updated by Nicolas CHARLES over 6 years ago

Status changed from Pending technical review to Pending release

Applied in changeset rudder-techniques|48cf123c4b84d9560eb857098b197158c0a7c862.

Actions

Copy link

#51

Updated by Vincent MEMBRÉ over 6 years ago

Status changed from Pending release to Released

This bug has been fixed in Rudder 4.1.13, 4.2.7 and 4.3.3 which were released today.

4.1.13: Announce Changelog
4.2.7: Announce Changelog
4.3.3: Announce Changelog
Download: https://www.rudder-project.org/site/get-rudder/downloads/

Actions

Copy link

Also available in: Atom PDF

Project

General

Profile

Rudder

Custom queries

Bug #6343

When a technique reports several time the exact same text, there is only one report generated

Updated by Nicolas CHARLES over 9 years ago

Updated by Vincent MEMBRÉ over 9 years ago

Updated by Benoît PECCATTE over 9 years ago

Updated by Benoît PECCATTE over 9 years ago

Updated by Vincent MEMBRÉ over 9 years ago

Updated by Nicolas CHARLES over 9 years ago

Updated by Benoît PECCATTE over 9 years ago

Updated by Vincent MEMBRÉ over 9 years ago

Updated by Vincent MEMBRÉ over 9 years ago

Updated by Vincent MEMBRÉ over 9 years ago

Updated by Vincent MEMBRÉ over 9 years ago

Updated by Vincent MEMBRÉ about 9 years ago

Updated by Vincent MEMBRÉ about 9 years ago

Updated by Vincent MEMBRÉ about 9 years ago

Updated by Vincent MEMBRÉ almost 9 years ago

Updated by Vincent MEMBRÉ almost 9 years ago

Updated by Vincent MEMBRÉ over 8 years ago

Updated by Vincent MEMBRÉ over 8 years ago

Updated by Vincent MEMBRÉ over 8 years ago

Updated by Vincent MEMBRÉ over 8 years ago

Updated by Vincent MEMBRÉ over 8 years ago

Updated by Vincent MEMBRÉ about 8 years ago

Updated by Vincent MEMBRÉ about 8 years ago

Updated by Vincent MEMBRÉ about 8 years ago

Updated by Vincent MEMBRÉ about 8 years ago

Updated by Vincent MEMBRÉ about 8 years ago

Updated by Vincent MEMBRÉ almost 8 years ago

Updated by Vincent MEMBRÉ almost 8 years ago

Updated by Jonathan CLARKE over 7 years ago

Updated by Benoît PECCATTE over 7 years ago

Updated by Vincent MEMBRÉ over 7 years ago

Updated by Vincent MEMBRÉ over 7 years ago

Updated by Vincent MEMBRÉ over 7 years ago

Updated by Benoît PECCATTE over 7 years ago

Updated by Vincent MEMBRÉ over 7 years ago

Updated by Alexis Mousset over 7 years ago

Updated by Alexis Mousset over 7 years ago

Updated by Vincent MEMBRÉ about 7 years ago

Updated by Vincent MEMBRÉ about 7 years ago

Updated by Vincent MEMBRÉ almost 7 years ago

Updated by Vincent MEMBRÉ almost 7 years ago

Updated by Vincent MEMBRÉ almost 7 years ago

Updated by Vincent MEMBRÉ over 6 years ago

Updated by Vincent MEMBRÉ over 6 years ago

Updated by Vincent MEMBRÉ over 6 years ago

Updated by Nicolas CHARLES over 6 years ago

Updated by Nicolas CHARLES over 6 years ago

Updated by Nicolas CHARLES over 6 years ago

Updated by Rudder Quality Assistant over 6 years ago

Updated by Nicolas CHARLES over 6 years ago

Updated by Vincent MEMBRÉ over 6 years ago