Bug #8837
closed
Makke uuidgen a single point of uuid generation
Description
/opt/rudder/bin/uuidgen: line 9: makeuuid: command not found
The uuidgen binary in /opt/rudder/bin should not ne balled uuigden since it does not have the same API as the read uuidgen binary.
Let's call it rudder-uuidgen.
Then it should be the only way to generate uuid, called by all other places that need. Thus it could contain all the smart thing we want to rudder the right uuid generation method depending on the situation.
Updated by Jonathan CLARKE over 8 years ago
- Target version set to 4.0.0~rc2
The file /opt/rudder/bin/uuidgen should not be installed on SLES. It should only be installed on AIX. This comes from the use of a single Makefile in master:
https://github.com/Normation/rudder-packages/blob/master/rudder-agent/SOURCES/Makefile#L189
Please also add a comment in the uuidgen file to explain this should only be used on AIX. And actually add a check, and if the OS has a uuidgen command use it, and only use our workaround if not.
Updated by Alexis Mousset over 8 years ago
- Subject changed from Missing makeuuid on sles 12 to Missing makeuuid on RPM systems
Updated by Alexis Mousset over 8 years ago
- Status changed from New to In progress
Updated by Alexis Mousset over 8 years ago
- Status changed from In progress to Pending technical review
- Assignee changed from Alexis Mousset to Jonathan CLARKE
- Pull Request set to https://github.com/Normation/rudder-packages/pull/1024
Updated by Benoît PECCATTE over 8 years ago
- Description updated (diff)
The uuidgen binary in /opt/rudder/bin should not ne balled uuigden since it does not have the same API as the read uuidgen binary.
Let's call it rudder-uuidgen.
Then it should be the only way to generate uuid, called by all other places that need it.
Updated by Benoît PECCATTE over 8 years ago
- Subject changed from Missing makeuuid on RPM systems to Makke uuidgen a single point of uuid generation
Updated by Benoît PECCATTE over 8 years ago
- Status changed from Pending technical review to In progress
- Assignee changed from Jonathan CLARKE to Benoît PECCATTE
Updated by Benoît PECCATTE over 8 years ago
- Status changed from In progress to Pending technical review
- Assignee changed from Benoît PECCATTE to Alexis Mousset
- Pull Request changed from https://github.com/Normation/rudder-packages/pull/1024 to https://github.com/Normation/rudder-packages/pull/1039
Updated by Benoît PECCATTE over 8 years ago
- Assignee changed from Alexis Mousset to Jonathan CLARKE
Updated by Benoît PECCATTE over 8 years ago
- Assignee changed from Jonathan CLARKE to Alexis Mousset
Updated by Benoît PECCATTE over 8 years ago
- Assignee changed from Alexis Mousset to Jonathan CLARKE
Updated by Benoît PECCATTE over 8 years ago
- Assignee changed from Jonathan CLARKE to Alexis Mousset
Updated by Benoît PECCATTE over 8 years ago
- Assignee changed from Alexis Mousset to Jonathan CLARKE
Updated by Benoît PECCATTE over 8 years ago
- Assignee changed from Jonathan CLARKE to Alexis Mousset
Updated by Benoît PECCATTE over 8 years ago
- Status changed from Pending technical review to Pending release
Applied in changeset rudder-packages|5b14b8fa28c38b07d701c996465b2c3fbc4334ea.
Updated by Benoît PECCATTE over 8 years ago
- Related to Bug #9040: uuidgen should not be source packaged anymore added
Updated by Benoît PECCATTE about 8 years ago
- Target version changed from 4.0.0~rc2 to 318
Updated by Vincent MEMBRÉ about 8 years ago
- Target version changed from 318 to 4.0.0~rc2
Updated by Vincent MEMBRÉ about 8 years ago
- Target version changed from 4.0.0~rc2 to 4.0.0~rc1
Updated by Alexis Mousset about 8 years ago
- Status changed from Pending release to Released
This bug has been fixed in Rudder 4.0.0 which was released the 10th November 2016.