Project

General

Profile

Bug #4499

Rudder init script kill all agent on Open VZ (or similar system)

Added by Nicolas CHARLES about 6 years ago. Updated almost 4 years ago.

Status:
Released
Priority:
1
Category:
Packaging
Target version:
Severity:
User visibility:
Effort required:
Priority:

Description

As reported by Alex in #4498, the init script is too eager in its agent killing

                # Presence of PID file
                # If the pid file is not readable or is empty, kill all process by name
                if [ ! -r ${CFENGINE_COMMUNITY_PID_FILE[$daemon]} -o ! -s ${CFENGINE_COMMUNITY_PID_FILE[$daemon]} ]
                then
                        # Try a killall
                        /usr/bin/killall -KILL ${CFENGINE_COMMUNITY_BIN[$daemon]}

On a openvz system, it would simply wipe all the daemon running

We should implement a similar fix as in http://www.rudder-project.org/redmine/issues/3928 / https://github.com/Normation/rudder-packages/pull/211/files to detect if its a VZ system, and kill only the relevant deamon

(as vzps.py is present only in 2.8, i'm targeting this ticket to 2.8)


Related issues

Related to Rudder - Bug #4498: Several issues with process management on Proxmox host (and container)RejectedActions
Related to Rudder - Bug #7381: Process management issues on nodes hosting LXC containersReleasedAlexis MOUSSETActions
#1

Updated by Nicolas CHARLES about 6 years ago

  • Project changed from Rudder to packages
#2

Updated by Vincent MEMBRÉ almost 6 years ago

  • Target version changed from 2.8.3 to 2.8.4
#3

Updated by Vincent MEMBRÉ almost 6 years ago

  • Target version changed from 2.8.4 to 133
#4

Updated by Vincent MEMBRÉ almost 6 years ago

  • Target version changed from 133 to 2.9.5
#5

Updated by Vincent MEMBRÉ over 5 years ago

  • Target version changed from 2.9.5 to 2.9.6
#6

Updated by Jonathan CLARKE over 5 years ago

  • Target version changed from 2.9.6 to 2.9.7
#7

Updated by Jonathan CLARKE over 5 years ago

  • Target version changed from 2.9.7 to 150
#8

Updated by Vincent MEMBRÉ over 5 years ago

  • Target version changed from 150 to 2.10.4
#9

Updated by Nicolas PERRON over 5 years ago

  • Target version changed from 2.10.4 to 2.6.18
#10

Updated by Nicolas PERRON over 5 years ago

  • Target version changed from 2.6.18 to 2.10.5
#11

Updated by Vincent MEMBRÉ over 5 years ago

  • Target version changed from 2.10.5 to 2.10.6
#12

Updated by Matthieu CERDA over 5 years ago

  • Target version changed from 2.10.6 to 2.10.7
#13

Updated by Vincent MEMBRÉ over 5 years ago

  • Target version changed from 2.10.7 to 2.10.8
#14

Updated by Vincent MEMBRÉ about 5 years ago

  • Target version changed from 2.10.8 to 2.10.9
#15

Updated by Nicolas CHARLES about 5 years ago

  • Target version changed from 2.10.9 to 2.11.6

Support for vzps is in branhc 2.11, so this ticket should be fixed in 2.11

#16

Updated by Nicolas CHARLES about 5 years ago

  • Assignee set to Benoît PECCATTE
#17

Updated by Vincent MEMBRÉ about 5 years ago

  • Target version changed from 2.11.6 to 2.11.7
#18

Updated by Vincent MEMBRÉ about 5 years ago

  • Target version changed from 2.11.7 to 2.11.8
#19

Updated by Vincent MEMBRÉ almost 5 years ago

  • Target version changed from 2.11.8 to 2.11.9
#20

Updated by Benoît PECCATTE almost 5 years ago

  • Project changed from packages to Rudder
  • Category set to Packaging
#21

Updated by Vincent MEMBRÉ almost 5 years ago

  • Target version changed from 2.11.9 to 2.11.10
#22

Updated by Vincent MEMBRÉ almost 5 years ago

  • Target version changed from 2.11.10 to 2.11.11
#23

Updated by Vincent MEMBRÉ almost 5 years ago

  • Target version changed from 2.11.11 to 2.11.12
#24

Updated by Vincent MEMBRÉ over 4 years ago

  • Target version changed from 2.11.12 to 2.11.13
#25

Updated by Vincent MEMBRÉ over 4 years ago

  • Target version changed from 2.11.13 to 2.11.14
#26

Updated by Vincent MEMBRÉ over 4 years ago

  • Target version changed from 2.11.14 to 2.11.15
#27

Updated by Vincent MEMBRÉ over 4 years ago

  • Target version changed from 2.11.15 to 2.11.16
#28

Updated by Vincent MEMBRÉ over 4 years ago

  • Target version changed from 2.11.16 to 2.11.17
#29

Updated by Vincent MEMBRÉ about 4 years ago

  • Target version changed from 2.11.17 to 2.11.18
#30

Updated by Vincent MEMBRÉ about 4 years ago

  • Target version changed from 2.11.18 to 2.11.19
#31

Updated by Alexis MOUSSET about 4 years ago

  • Related to Bug #7381: Process management issues on nodes hosting LXC containers added
#32

Updated by Alexis MOUSSET about 4 years ago

  • Status changed from New to In progress
  • Assignee changed from Benoît PECCATTE to Alexis MOUSSET
#33

Updated by Alexis MOUSSET about 4 years ago

  • Status changed from In progress to Pending technical review
  • Pull Request set to https://github.com/Normation/rudder-packages/pull/879
#34

Updated by Alexis MOUSSET about 4 years ago

  • Assignee changed from Alexis MOUSSET to Benoît PECCATTE
#35

Updated by Alexis MOUSSET about 4 years ago

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

Updated by Vincent MEMBRÉ almost 4 years ago

  • Status changed from Pending release to Released

This bug has been fixed in Rudder 2.11.19, 3.0.14, 3.1.8 and 3.2.1 which were released today.

Also available in: Atom PDF