Bug #10498: Stopping slapd fails if pid file is empty - Rudder - Issue Tracker

Actions

Copy link

Bug #10498

closed

Stopping slapd fails if pid file is empty

Added by Janos Mattyasovszky about 7 years ago. Updated about 6 years ago.

Status:

Released

Priority:

N/A

Assignee:

Benoît PECCATTE

Category:

System integration

Target version:

4.1.10

Pull Request:

https://github.com/Normation/rudder-p...

Severity:

Minor - inconvenience | misleading | easy workaround

UX impact:

User visibility:

Operational - other Techniques | Technique editor | Rudder settings

Effort required:

Small

Priority:

Name check:

Fix check:

Regression:

Description

If the pid file for slapd is empty, the init script fails.
Here is the trace:

+ message info '[INFO] Halting OpenLDAP...'
+ '[' Linux = Linux ']'
+ logger -p local4.info -s -t rudder-slapd -i '[INFO] Halting OpenLDAP...'
<166>Mar 24 07:34:57 rudder-slapd[1394]: [INFO] Halting OpenLDAP...
+ '[' '!' -r /var/rudder/run/slapd.pid ']'
++ cat /var/rudder/run/slapd.pid
+ PID=
+ kill -INT
kill: usage: kill [-s sigspec | -n signum | -sigspec] pid | jobspec ... or kill -l [sigspec]
+ i=0
+ '[' -e /proc/ ']'
+ '[' 0 -eq 30 ']'
++ expr 0 + 1
+ i=1
[...]

Note: slapd was actually running.

# pgrep -f /opt/rudder/libexec/slapd
17492

After writing the PID to the file it did stop.

Found in: rudder-inventory-ldap-4.1.0.rc2.git201703230121-1.SLES.12.x86_64

Subtasks 1 (0 open — 1 closed)

Actions

Copy link

Updated by François ARMAND about 7 years ago

Severity set to Minor - inconvenience | misleading | easy workaround
User visibility set to Operational - other Techniques | Technique editor | Rudder settings

Actions

Copy link

Updated by Benoît PECCATTE about 7 years ago

Priority set to 17

Actions

Copy link

Updated by Alexis Mousset almost 7 years ago

Category set to System integration

Actions

Copy link

Updated by Benoît PECCATTE almost 7 years ago

Priority changed from 17 to 16

Actions

Copy link

Updated by Benoît PECCATTE almost 7 years ago

Priority changed from 16 to 31

Actions

Copy link

Updated by Benoît PECCATTE almost 7 years ago

Effort required set to Small
Priority changed from 31 to 35

Actions

Copy link

Updated by Benoît PECCATTE over 6 years ago

Priority changed from 35 to 43

Actions

Copy link

Updated by Benoît PECCATTE about 6 years ago

Target version set to 4.1.10
Priority changed from 43 to 41

Actions

Copy link

Updated by Benoît PECCATTE about 6 years ago

Status changed from New to In progress
Assignee set to Benoît PECCATTE

Actions

Copy link

#10

Updated by Benoît PECCATTE about 6 years ago

Status changed from In progress to Pending technical review
Assignee changed from Benoît PECCATTE to Alexis Mousset
Pull Request set to https://github.com/Normation/rudder-packages/pull/1503

PR https://github.com/Normation/rudder-packages/pull/1503

Actions

Copy link

#11