Bug #3170
closedTechniques 'Zypper Package Manager Configuration' and 'RUG / YaST package manager configuration (ZMD)': Broken Reporting when configuration more than one repository which should not be added
Description
When using the Technique Zypper Package Manager Configuration, it seems that the only way to have a successful reporting is to configure no more than one repository and which should not be added into zypper.
Here are the differents behaviour:- If one repository is set and should be added => All reporting of Rudder will be in an Unkown state
- If one repository is set and should not be added => Success
- If several repositories are set and should be added => reporting about the Directive is in a No Response state and no report appears in the Technical log of the node about this Directive
- If several repositories are set and should not be added => reporting about the Directive is in a No Response state and no report appears in the Technical log of the node about this Directive
Pull Request URL: https://github.com/Normation/rudder-techniques/pull/59 https://github.com/Normation/rudder-techniques/pull/46
Edit: This bug is present in the Technique 'RUG / YaST package manager configuration (ZMD)' too.
Files
Updated by Nicolas PERRON almost 12 years ago
- Category set to Techniques
- Assignee set to Nicolas PERRON
- Priority changed from N/A to 1 (highest)
- Target version set to 2.3.10
Updated by Nicolas PERRON almost 12 years ago
Here is an extract from tail -f /var/log/rudder/reports/all.log | grep Repository on the server with two Repositories to set:
2013-01-10T16:04:23+01:00 sles-11-sp1-64 rudder[27535]: R: @@zypperPackageManagerSettings@@result_success@@f6061429-1565-4373-9a6f-e9c253a9cc91@@98874df5-9099-4632-bea5-075154ae4361@@6f6061429-1565-4373-9a6f-e9c253a9cc91@@98874df5-9099-4632-bea5-075154ae4361@@6@@Repository@@Rudder Project@@2013-01-10 16:04:17+01:00##59e66fe7-7f0e-497d-8055-a103116b4a08@#The Zypper source Rudder Project was already here. Skipping... 2013-01-10T16:04:23+01:00 sles-11-sp1-64 rudder[27535]: R: @@zypperPackageManagerSettings@@result_success@@f6061429-1565-4373-9a6f-e9c253a9cc91@@98874df5-9099-4632-bea5-075154ae4361@@6f6061429-1565-4373-9a6f-e9c253a9cc91@@98874df5-9099-4632-bea5-075154ae4361@@6@@Repository@@Rudder Project Unstable@@2013-01-10 16:04:17+01:00##59e66fe7-7f0e-497d-8055-a103116b4a08@#The Zypper source Rudder Project Unstable was already here. Skipping...
Updated by Nicolas PERRON almost 12 years ago
The bug seems to come from a misuse of a the Technique TrackingKey. Here is a fix for branch 2.3
Updated by Nicolas PERRON over 11 years ago
- Description updated (diff)
- Status changed from New to Pending technical review
- % Done changed from 0 to 100
Pull Request URL added: https://github.com/Normation/rudder-techniques/pull/46
Updated by Nicolas PERRON over 11 years ago
- Assignee changed from Nicolas PERRON to Jonathan CLARKE
Jon, could you review this pull request, please ?
Updated by Nicolas PERRON over 11 years ago
- Status changed from Pending technical review to Discussion
I suppose the pull request can be cancelled.
The reporting can't work since the Tracking Key used is based on a multivalued component which can be null:- If the tracking key is null, Rudder will not be aware of relations between Directives and reports received.
- There is no any chance to have reports on the none multivalued components
- One Technique which will do the job for adding repositories
- Another Technique which will set general settings for zypper (install recommends)
- Zypper package manager configuration rename into Zypper repositories management
- Add Zypper settings management
Jon, what do you think about ?
Updated by Jonathan CLARKE over 11 years ago
Nicolas PERRON wrote:
I suppose the pull request can be cancelled.
No need for that, just update it.
The reporting can't work since the Tracking Key used is based on a multivalued component which can be null:The solution is to split this Technique into two:
- If the tracking key is null, Rudder will not be aware of relations between Directives and reports received.
- There is no any chance to have reports on the none multivalued components
I wonder: shouldn't we rename the Technique if the goals are altered ?
- One Technique which will do the job for adding repositories
- Another Technique which will set general settings for zypper (install recommends)
- Zypper package manager configuration rename into Zypper repositories management
Yep, sounds good.
- Add Zypper settings management
I'd prefer "Zypper package manager settings".
Updated by Matthieu CERDA over 11 years ago
- Target version changed from 2.3.10 to 2.3.11
Updated by Nicolas PERRON over 11 years ago
- Subject changed from Technique Zypper Package Manager Configuration: Broken Reporting when configuration more than one repository which should not be added to Techniques 'Zypper Package Manager Configuration' and 'RUG / YaST package manager configuration (ZMD)': Broken Reporting when configuration more than one repository which should not be added
- Description updated (diff)
- Status changed from Discussion to Pending technical review
- Assignee changed from Jonathan CLARKE to Nicolas CHARLES
Updated the title of the ticket. This bug appears not only on 'Zypper Package Manager Configuration' but with 'RUG / YaST package manager configuration (ZMD) too.
Pull Request URL updated: https://github.com/Normation/rudder-techniques/pull/59
Nicolas, could you review it please ?
Updated by Nicolas PERRON over 11 years ago
Nicolas PERRON wrote:
Updated the title of the ticket. This bug appears not only on 'Zypper Package Manager Configuration' but with 'RUG / YaST package manager configuration (ZMD) too.
Pull Request URL updated: https://github.com/Normation/rudder-techniques/pull/59
Nicolas, could you review it please ?
Reuse and update previous Pull Request instead: https://github.com/Normation/rudder-techniques/pull/46
Nicolas, could you review it, please ?
Updated by Nicolas PERRON over 11 years ago
- Description updated (diff)
Pull Request URL updated on description.
Updated by Nicolas PERRON over 11 years ago
- Status changed from Pending technical review to Pending release
Applied in changeset commit:f7cb8c5f1357e9d064d93ae85e914e4c83177532.
Updated by Nicolas CHARLES over 11 years ago
Applied in changeset commit:3025177c70f40490de20b0938e4eb456e2d5db0e.
Updated by Nicolas PERRON over 11 years ago
Applied in changeset commit:98701b512e2dd63ca982dc6d0851a031277e0bc5.
Updated by Nicolas PERRON over 11 years ago
Applied in changeset commit:ba2d070f49fd3a720679c23e2040c67993c3600b.
Updated by Nicolas PERRON over 11 years ago
Applied in changeset commit:f6eed4b11b64fdcda160cc066c8b2ab20c783986.
Updated by Matthieu CERDA over 11 years ago
- Status changed from Pending release to Released
This ticket has been adressed in the 2.3.11 release of Rudder.
The ChangeLog is available here: http://www.rudder-project.org/foswiki/bin/view/System/Documentation:ChangeLog23
Updated by Benoît PECCATTE over 9 years ago
- Project changed from 24 to Rudder
- Category changed from Techniques to Techniques