Bug #4147
closedFirst inventory sending may not contains UUID
Description
When we install for the first time rudder-agent, it created the uuid.hive file with no UUID only a commentary that the uuid will be generated.
We need to enfore creation of the uuid with uuidgen if this is a first installation.
This may lead to inventory sending with a wrong UUID (the comment we put in the file)
I need to look in 2.4 if this could happen
Updated by Vincent MEMBRÉ almost 11 years ago
- Status changed from New to Pending technical review
- Assignee changed from Vincent MEMBRÉ to Jonathan CLARKE
- Pull Request set to https://github.com/Normation/rudder-packages/pull/185
Pull request here: https://github.com/Normation/rudder-packages/pull/185
Updated by Vincent MEMBRÉ almost 11 years ago
Is there a reason why we define CFRUDDER_FIRST_INSTALL if there is a way for each packaging system to determine if it's an upgrade or a first install?
For deb, See 2nd answer here : http://askubuntu.com/questions/101962/script-to-only-execute-during-first-install-of-a-package
For RPM look at this link (section Deal with upgrade) http://wiki.mandriva.com/en/Development/Howto/RPM
Updated by Jonathan CLARKE almost 11 years ago
- Status changed from Pending technical review to Discussion
- Assignee changed from Jonathan CLARKE to Vincent MEMBRÉ
I understand and agree with the proposed fix.
But I am absolutely sure that there was already a uuidgen can in these post inst scripts. Can you please look through the got log and find where (which commit, which ticket) this was removed?
Updated by Vincent MEMBRÉ almost 11 years ago
It was related to ticket #3925, specially the commit commit:d4d3097ee9d4714c87efe9bbe22ab0dc014052f0
This has been reverted by commit:d4d3097ee9d4714c87efe9bbe22ab0dc014052f0 then reapplied commit:a01181b20a0f08700f5cfaae6a483f4ef2813aeb (same issue treated, so it mostly keeps the same logic than in first commit)
I have another idea about this fix: What do you think of running /opt/rudder/bin/check-rudder-agent in every postinst ?
I just got the idea but it check the uuid and could even restore a backup. however I still need to check its other actions.
Updated by Vincent MEMBRÉ almost 11 years ago
- Assignee changed from Vincent MEMBRÉ to Jonathan CLARKE
Updated by Vincent MEMBRÉ almost 11 years ago
In fact check-rudder-agent also mess with cfengine process, which we do not want here.
Here we need more to focus on the uuid and keep it simple (first install -> always generate an uuid)
Updated by Vincent MEMBRÉ almost 11 years ago
- Status changed from Discussion to Pending technical review
After discution with Jon we decided to use the /opt/rudder/bin/check-rudder-agent
I realized that it was already done in commit:d4d3097ee9d4714c87efe9bbe22ab0dc014052f0 and reverted by commit:d1c61325c4eb817f8c051ca474548e0a9bcf70a0 and then not added again in commit: a01181b20a0f08700f5cfaae6a483f4ef2813aeb
I updated the pull request with new correction: https://github.com/Normation/rudder-packages/pull/185
Updated by Vincent MEMBRÉ almost 11 years ago
- Status changed from Pending technical review to Pending release
- % Done changed from 0 to 100
Applied in changeset commit:3d74cf3fb5bf4c0b5a38e8cb566905315780ff8b.
Updated by Jonathan CLARKE almost 11 years ago
Applied in changeset commit:7e10dbdb746c8cf5d1ca8375857c6194efd45aac.
Updated by Nicolas PERRON almost 11 years ago
- Status changed from Pending release to Released
This bug has been fixed in Rudder 2.4.12 which was released today.
Check out:
- The release announcement: http://www.rudder-project.org/pipermail/rudder-announce/2013-November/000061.html
- The full ChangeLog: http://www.rudder-project.org/foswiki/bin/view/System/Documentation:ChangeLog24
- Download information: https://www.rudder-project.org/site/get-rudder/downloads/
Updated by Benoît PECCATTE over 9 years ago
- Project changed from 34 to Rudder
- Category set to Packaging