Bug #3146: remove : crontab not clean - Rudder - Issue Tracker

Actions

Copy link

Bug #3146

closed

remove : crontab not clean

Added by Francois BAYART over 11 years ago. Updated about 9 years ago.

Status:

Released

Priority:

Assignee:

Jonathan CLARKE

Category:

Packaging

Target version:

2.5.0~rc1

Pull Request:

Severity:

UX impact:

User visibility:

Effort required:

Priority:

Name check:

Fix check:

Regression:

Description

after my "apt-get remove rudder-agent && dpkg --purge rudder-agent", I have kill the cfengine process but after 5 minutes its come back !

The /etc/crontab have this line :

0,5,10,15,20,25,30,35,40,45,50,55 * * * * root if [ `ps -efww | grep cf-execd | grep "/var/rudder/cfengine-community/bin/cf-execd" 
| grep -v grep | wc -l` -eq 0 ]; then /var/rudder/cfengine-community/bin/cf-execd; fi

Subtasks 1 (0 open — 1 closed)

Related issues 2 (0 open — 2 closed)

Actions

Copy link

Updated by François ARMAND over 11 years ago

Status changed from New to 8
Assignee set to Matthieu CERDA
Priority changed from N/A to 3
Target version set to 2.3.10

Actions

Copy link

Updated by Jonathan CLARKE over 11 years ago

Priority changed from 3 to 1
Target version changed from 2.3.10 to 2.5.0~rc1

Matthieu, can you work on a change for this for 2.5 please? This is too impacting to change in 2.3 or 2.4 (stable versions), but can make it into 2.5.

Actions

Copy link

Updated by Francois BAYART over 11 years ago

same remark with :
- /etc/apt/apt.conf.d/99rudder
- /etc/rsyslog.d/rudder-agent.conf

Actions

Copy link

Updated by Jonathan CLARKE over 11 years ago

We've decided to move the cron line as suggested by Fabrice to /etc/cron.d/rudder-agent. Steps to complete:

Create this file in the system Technique common (the plain cron file to be copied as is)
Edit the system Technique common to copy this file to /etc/cron.d/rudder-agent
Edit the system Technique common to delete_lines the old line from /etc/crontab
Edit the packaging of rudder-agent so that SOURCES/Makefile copies the rudder-agent cron file from the rudder-techniques repo
Adapt the packaging (deb and RPM) to copy this file into place on install, and clear it up on package purge

Actions

Copy link

Updated by Matthieu CERDA over 11 years ago

Status changed from 8 to In progress
Assignee changed from Matthieu CERDA to Jonathan CLARKE

The first part has been done: a Technique update that puts the Rudder cron configuration in cron.d instead of crontab.

There is still the packaging work to do.

A little word of caution, in the Technique, I used the local_cp body (defined in the stdlib), we might want to use another one as I suspect that it uses the mtime file comparison method by default. (Should not harm, but might cause some funny things if the user alters the file on the destination machine...)

Jon, you turn :)

Actions

Copy link