Bug #7623: Unexpected reports using "rpmPackageInstallation" technique - 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 #7623

closed

Unexpected reports using "rpmPackageInstallation" technique

Added by Jonathan CLARKE almost 9 years ago. Updated almost 9 years ago.

Status:

Released

Priority:

N/A

Assignee:

Nicolas CHARLES

Category:

Techniques

Target version:

2.11.18

Pull Request:

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

Severity:

UX impact:

User visibility:

Effort required:

Priority:

Name check:

Fix check:

Regression:

Description

Sometime both a repaired and a success message can come through. This is due to poor checking of the result classes.

History
Notes
Property changes
Associated revisions

Actions

Copy link

Updated by Jonathan CLARKE almost 9 years ago

Status changed from New to In progress
Assignee set to Jonathan CLARKE

Actions

Copy link

Updated by Jonathan CLARKE almost 9 years ago

Translation missing: en.field_tag_list set to Quick and important

Actions

Copy link

Updated by Benoît PECCATTE almost 9 years ago

Assignee changed from Jonathan CLARKE to Benoît PECCATTE

Actions

Copy link

Updated by Benoît PECCATTE almost 9 years ago

Status changed from In progress to Discussion
Assignee changed from Benoît PECCATTE to Jonathan CLARKE

There are 12 versions of the technique in 2.11 which are all broken in a different way.
This doesn't seem quick for me.

Actions

Copy link

Updated by Jonathan CLARKE almost 9 years ago

Status changed from Discussion to Pending technical review
Assignee changed from Jonathan CLARKE to Nicolas CHARLES
Pull Request set to https://github.com/Normation/rudder-techniques/pull/840

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

Actions

Copy link

Updated by Jonathan CLARKE almost 9 years ago

Benoît PECCATTE wrote:

There are 12 versions of the technique in 2.11 which are all broken in a different way.
This doesn't seem quick for me.

Actually they are all broken in the exact same way, for this bug. I wrote the fix once and copy/pasted it to all versions - except the latest, which uses our shared reporting library, so I just changed that to use the "generic" method, that avoids this bug anyway.

Actions

Copy link