Project

General

Profile

Actions

Bug #10860

closed

Backup of apache vhost by rudder-server-relay created in bad directory

Added by Janos Mattyasovszky almost 7 years ago. Updated over 6 years ago.

Status:
Released
Priority:
N/A
Category:
Packaging
Target version:
Severity:
Minor - inconvenience | misleading | easy workaround
UX impact:
User visibility:
Operational - other Techniques | Technique editor | Rudder settings
Effort required:
Very Small
Priority:
48
Name check:
Fix check:
Regression:

Description

The package 4.1.3 of rudder-server-relay creates backups of old files in plain /var/backups, while all other packages are using /var/backups/rudder:

# Move old virtual hosts out of the way
for OLD_VHOST in rudder-default rudder-default-ssl rudder-default.conf rudder-default-ssl.conf rudder-vhost.conf rudder-vhost-ssl.conf rudder-relay-vhost.conf
 rudder-relay-vhost-ssl.conf; do
        if [ -f /etc/apache2/vhosts.d/${OLD_VHOST} ]; then
                echo -n "INFO: An old rudder virtual host file has been detected (${OLD_VHOST}), it will be moved to /var/backups." 
                mkdir -p /var/backups
                mv /etc/apache2/vhosts.d/${OLD_VHOST} /var/backups/${OLD_VHOST}-$(date +%s)
                echo " Done" 
        fi
done

These are the paths that have /var/backups in the rpm's scripts:

# rpm -q --scripts -a rudder-* | grep -o '/var/backups/[^ ]*' | sort -u
/var/backups/${OLD_VHOST}-$(date                            <==== This is the only one not pointing to the rudder subdir.
/var/backups/rudder
/var/backups/rudder" 
/var/backups/rudder/cfengine_policy_server.dat-$(date
/var/backups/rudder/policy_server.dat-$(date
/var/backups/rudder/ppkeys-$(date
/var/backups/rudder/rudder-agent.$(basename
/var/backups/rudder/uuid-$(date

Is this behavior on purpose or simply by mistake?

Actions

Also available in: Atom PDF