Bug #3862: The script rudder-upgrade checked a wrong apache config file for SuSE and RHEL/CentOS - Rudder - Issue Tracker

Actions

Copy link

Bug #3862

closed

The script rudder-upgrade checked a wrong apache config file for SuSE and RHEL/CentOS

Added by Olivier Mauras over 11 years ago. Updated almost 10 years ago.

Status:

Released

Priority:

Assignee:

Jonathan CLARKE

Category:

Packaging

Target version:

2.7.1

Pull Request:

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

Severity:

UX impact:

User visibility:

Effort required:

Priority:

Name check:

Fix check:

Regression:

Description

When rudder-upgrade is run it outputs the following error:

/opt/rudder/bin/rudder-upgrade
INFO: Checking LDAP service status... OK
INFO: Checking PostgreSQL service status... OK
WARNING: Some event log are still based on an old file format (file format 1), please upgrade first to 2.6 to make this migration
INFO: Modifying system group entries in LDAP if necessary... Done
grep: /etc/httpd/conf.d/rudder-default: No such file or directory
INFO: Adding Apache alias for User Manual... sed: can't read /etc/httpd/conf.d/rudder-default: No such file or directory

Line 99 needs to be: RUDDER_WWW_CONF="${WWW_DIR}/rudder-default.conf"

Actions

Copy link

Updated by Nicolas PERRON over 11 years ago

Project changed from 24 to 34
Status changed from New to In progress
Assignee set to Nicolas PERRON

Thanks for you report !
I'm on it

Actions

Copy link

Updated by Nicolas PERRON over 11 years ago

Argh... why on Debian the file is rudder-default and on RPM the file is rudder-default*.conf* ?!

I wonder if I should adapt the rudder-upgrade script or set the same file for all the OS..

Actions

Copy link

Updated by Nicolas PERRON over 11 years ago

Assignee changed from Nicolas PERRON to Matthieu CERDA
% Done changed from 0 to 100
Pull Request set to https://github.com/Normation/rudder-packages/pull/101

I'v made a Pull Request: https://github.com/Normation/rudder-packages/pull/101

I've adapted the rudde-upgrade file. This is with less consequence IMHO

Matthieu, could you review it please ?

Actions

Copy link

Updated by Nicolas PERRON over 11 years ago

Status changed from In progress to Pending technical review
Assignee changed from Matthieu CERDA to Jonathan CLARKE

Jon, could you review this PR please ?

Actions

Copy link

Updated by Nicolas PERRON over 11 years ago

Status changed from Pending technical review to Pending release

Applied in changeset commit:424e276bd50d7c246dce167f6f86f43c75a74c2c.

Actions

Copy link

Updated by Jonathan CLARKE over 11 years ago

Applied in changeset commit:158b1c7f45a068ccd9f59990568b566c6baf01f7.

Actions

Copy link

Updated by Nicolas PERRON over 11 years ago

Subject changed from rudder-upgrade apache config file invalid name to The script rudder-upgrade checked a wrong apache config file for SuSE and RHEL/CentOS

Actions

Copy link