Actions
Bug #6051
closedWarning after policy generation, /bin/pidof does not exists
Status:
Released
Priority:
1 (highest)
Assignee:
Matthieu CERDA
Category:
System integration
Target version:
Pull Request:
Severity:
UX impact:
User visibility:
Effort required:
Priority:
Name check:
Fix check:
Regression:
Description
After updating my promises, I got this message:
[2014-12-30 02:32:20] WARN com.normation.rudder.services.policies.RudderCf3PromisesFileWriterServiceImpl - Failed to reload CFEngine server promises with "command /usr/bin/killall -SIGHUP cf-serverd || /bin/kill -SIGHUP `/bin/pidof cf-serverd 2>/dev/null`" - cause is `/bin/pidof: No such file or directory
On SLES11 and RHEL6, pidof path is /sbin/pidof ...
Updated by Nicolas CHARLES almost 10 years ago
- Assignee changed from Nicolas CHARLES to Matthieu CERDA
Updated by Matthieu CERDA almost 10 years ago
So, after testing and discussing with our local Scala experts, I see that executing the command to reload the server during policy generation is done "execve" style in Scala, meaning it takes ONE command and its arguments,
The best approach I can come with for this is to just delegate all the magic to a dedicated script, for several reasons:- No more fiddling with process execution magic
- One script to do it, and do it well instead of an unreadable line in the .properties (we've done this before in cron, never again...)
- Change in rudder the command to be executed in the sample properties(and correct a small but horribly frustrating logging typo)
- Adapt the packaging to add the right line during migration, and package the script itself
Updated by Nicolas CHARLES almost 10 years ago
- Target version changed from 3.0.0 to 2.11.6
Caution, this appeared in 2.11, so it need to be fixed there
Updated by Matthieu CERDA almost 10 years ago
- Status changed from New to Pending technical review
Updated by Vincent MEMBRÉ almost 10 years ago
- Status changed from Pending technical review to Pending release
- Pull Request set to https://github.com/Normation/rudder-packages/pull/576
This was fixed in #6055, I moved back the PR and the commit from there as 6051 is the parent issue.
Since this is a bug, it does not need to have a sub issue to be fixed
Updated by Vincent MEMBRÉ almost 10 years ago
- Target version changed from 2.11.6 to 2.11.7
Updated by Vincent MEMBRÉ almost 10 years ago
- Status changed from Pending release to Released
This bug has been fixed in Rudder 2.11.7, which was released these days.
- Announcement 2.11
- Changelog 2.11
- Download information: https://www.rudder-project.org/site/get-rudder/downloads/
Updated by Benoît PECCATTE about 9 years ago
- Pull Request deleted (
https://github.com/Normation/rudder-packages/pull/576)
Actions