Project

General

Profile

Actions

Bug #9950

closed

User story #8353: Implement notifications for different server-side actions and events (hooks)

Policy generation fails in 4.1 with 'kill: usage: kill [-s sigspec | -n signum | -sigspec] pid | jobspec ... or kill -l [sigspec]'

Added by Nicolas CHARLES about 7 years ago. Updated about 7 years ago.

Status:
Released
Priority:
N/A
Category:
System integration
Target version:
Severity:
UX impact:
User visibility:
Effort required:
Priority:
Name check:
Fix check:
Regression:

Description

When doing a policy generation on Centos6, I get the following error

⇨ Policy update error for process '3' at 2017-01-11 09:35:42
⇨ Exit code=1 for hook: '/opt/rudder/etc/hooks.d/policy-generation-finished/50-sigup-cf-serverd HookParameter(RUDDER_GENERATION_DATETIME,2017-01-11T09:35:39.234Z) HookParameter(RUDDER_END_GENERATION_DATETIME,2017-01-11T09:35:42.619Z) HookParameter(RUDDER_NODEIDS,root) HookParameter(RUDDER_NUMBER_NODES_UPDATED,1) HookParameter(RUDDER_ROOT_POLICY_SERVER_UPDATED,0)'. Stdout: '' Stderr: 'which: no killall in ((null)) kill: usage: kill [-s sigspec | -n signum | -sigspec] pid | jobspec ... or kill -l [sigspec] '

Issue is: it calls /opt/rudder/etc/hooks.d/policy-generation-finished/50-sigup-cf-serverd, which calls /opt/rudder/bin/rudder-reload-cf-serverd

Somehow, /opt/rudder/bin/rudder-reload-cf-serverd doesn't know that "killall" is there (probably no PATH available for the webapp)), so it falls back to

kill -1 $(pidof cf-serverd 2>/dev/null)

however, pidof cf-serverd returns nothing if it not running, so kill fails, and worse, pidof isn't in the PATH either

Replacing pidof by it path or killall by its path solves the issue, but I'm pretty sure it should rather have the PATH defined

Actions #1

Updated by Benoît PECCATTE about 7 years ago

  • Assignee changed from Alexis Mousset to François ARMAND

This is what happed when the PATH is not defined.

We should have inherited the proper environment to be able to run a script.

Actions #2

Updated by François ARMAND about 7 years ago

  • Status changed from New to In progress
Actions #3

Updated by François ARMAND about 7 years ago

  • Status changed from In progress to Pending technical review
  • Assignee changed from François ARMAND to Vincent MEMBRÉ
  • Pull Request set to https://github.com/Normation/rudder/pull/1445
Actions #4

Updated by François ARMAND about 7 years ago

  • Assignee changed from Vincent MEMBRÉ to Alexis Mousset
Actions #5

Updated by François ARMAND about 7 years ago

  • Assignee changed from Alexis Mousset to Vincent MEMBRÉ
Actions #6

Updated by François ARMAND about 7 years ago

  • Status changed from Pending technical review to Pending release
  • % Done changed from 0 to 100
Actions #7

Updated by Vincent MEMBRÉ about 7 years ago

  • Parent task set to #8353
Actions #8

Updated by Vincent MEMBRÉ about 7 years ago

  • Status changed from Pending release to Released

This bug has been fixed in Rudder 4.1.0 which was released today.

Actions

Also available in: Atom PDF