Project

General

Profile

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 over 9 years ago.

Status:
Released
Priority:
2
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:

  1. /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
 #1

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
 #2

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
 #3

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
 #4

Updated by Nicolas PERRON about 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
 #5

Updated by Nicolas PERRON about 11 years ago

  • Status changed from Pending technical review to Pending release

Applied in changeset commit:424e276bd50d7c246dce167f6f86f43c75a74c2c.

Actions
Copy link
 #6

Updated by Jonathan CLARKE about 11 years ago

Applied in changeset commit:158b1c7f45a068ccd9f59990568b566c6baf01f7.

Actions
Copy link
 #7

Updated by Nicolas PERRON about 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
 #8

Updated by Nicolas PERRON about 11 years ago

  • Status changed from Pending release to Released

This bug has been fixed in Rudder 2.7.1, which was released today.
Check out:

Actions
Copy link
 #9

Updated by Benoît PECCATTE over 9 years ago

  • Project changed from 34 to Rudder
  • Category set to Packaging
Actions
Copy link

Also available in: Atom PDF