User story #4910
closed
User story #4740: Have a rudder-agent package that does not bundle fusioninventory
Have a rudder-agent package that does not bundle fusioninventory (integration)
Added by Matthieu CERDA over 10 years ago.
Updated over 9 years ago.
Description
See the upper ticket for details :)
- Status changed from In progress to Pending technical review
- Assignee changed from Matthieu CERDA to Jonathan CLARKE
- % Done changed from 0 to 100
- Pull Request set to https://github.com/Normation/rudder-packages/pull/332
- Status changed from Pending technical review to Pending release
Applied in changeset commit:807e87efa48e370d3d43ca177f6aa3cbbc1323f5.
Applied in changeset commit:080861366b94195aef830103f152b6d50b8811a6.
- Status changed from Pending release to Discussion
- Assignee changed from Jonathan CLARKE to Matthieu CERDA
Guys, how did this get merged? This is absolutely not OK for a technical review. I don't understand how this could have been passed.
Main issue: duplication. There is absolutely no way we are going to maintain duplicate copies of all of these files. I didn't even look any further.
Please revert and propose an implementation that respects our quality rules.
- Status changed from Discussion to In progress
- % Done changed from 100 to 60
- Pull Request deleted (
https://github.com/Normation/rudder-packages/pull/332)
Commit reverted, let's get back to work :)
- Status changed from In progress to Pending technical review
- Assignee changed from Matthieu CERDA to Jonathan CLARKE
- % Done changed from 60 to 100
- Pull Request set to https://github.com/Normation/rudder-packages/pull/340
- Status changed from Pending technical review to Pending release
Applied in changeset commit:14d7fa5ff937d927c448a8d3ae42e25cecffed45.
Applied in changeset commit:47240c3726978ec4b2f38c1bdc24b90fe914c673.
- Status changed from Pending release to Released
This bug has been fixed in Rudder 2.11.0~beta1 that was released yesterday.
- Category set to Packaging
- Tracker changed from 6 to User story
Also available in: Atom
PDF