Bug #3075: During a migration from 2.4.0~rc1 to 2.4.0~rc2, rudder-inventory-ldap postinst and rudder-upgrade script display two disturbing messages about LDAP and rudder-web.properties - Rudder - Issue Tracker

Actions

Copy link

Bug #3075

closed

During a migration from 2.4.0~rc1 to 2.4.0~rc2, rudder-inventory-ldap postinst and rudder-upgrade script display two disturbing messages about LDAP and rudder-web.properties

Added by Nicolas PERRON over 11 years ago. Updated about 9 years ago.

Status:

Released

Priority:

Assignee:

Nicolas PERRON

Category:

Packaging

Target version:

2.4.0

Pull Request:

Severity:

UX impact:

User visibility:

Effort required:

Priority:

Name check:

Fix check:

Regression:

Description

Pull Request URL: https://github.com/Normation/rudder-packages/pull/8

$ sudo aptitude install rudder-server-root
The following packages will be upgraded: 
  rudder-agent rudder-inventory-endpoint rudder-inventory-ldap rudder-jetty rudder-reports rudder-server-root rudder-techniques rudder-webapp 
8 packages upgraded, 0 newly installed, 0 to remove and 9 not upgraded.
Need to get 137 MB of archives. After unpacking 32.8 kB will be used.
Do you want to continue? [Y/n/?] y
[...]
Setting up rudder-inventory-ldap (2.4.0~rc2-squeeze0) ...
Stopping enhanced syslogd: rsyslogd.
Starting enhanced syslogd: rsyslogd.
0a1
> activeTechniqueId
2a4
> directiveId
6a9,11
> machineId
> nodeGroupId
> nodeId
7a13
> ruleId
8a15
> softwareId
9a17,18
> techniqueCategoryId
> techniqueId
OpenLDAP indexes are not up to date, reindexing...
[...]
/opt/rudder/etc/rudder-users.xml is conform
<configuration><appender name="STDOUT" class="ch.qos.logback.core.ConsoleAppender"><encoder class="ch.qos.logback.classic.encoder.PatternLayoutEncoder"><Pattern>%d{[yyyy-MM-dd HH:mm:ss]} %-5level %logger - %msg%n%xEx{0}</Pattern></encoder></appender><property name="OPSLOG_DIR" value="/var/log/rudder/core" /><appender name="OPSLOG" class="ch.qos.logback.core.FileAppender"><file>${OPSLOG_DIR}/rudder-webapp.log</file><append>true</append><encoder><pattern>%d{MMM dd HH:mm:ss} ${HOSTNAME} rudder[%logger]: [%level] %msg%n</pattern></encoder></appender><root level="info"><appender-ref ref="STDOUT" /></root><logger name="trace.ldif.in.file" level="off" /><logger name="comet_trace" level="info" /><logger name="org.springframework" level="warn" /><logger name="net.liftweb.util.TimeHelpers" level="warn" /><logger name="report" level="info" additivity="false"><appender-ref ref="OPSLOG" /><appender-ref ref="STDOUT" /></logger><logger name="application" level="info" additivity="false"><appender-ref ref="OPSLOG" /><appender-ref ref="STDOUT" /></logger><logger name="migration" level="info" additivity="false"><appender-ref ref="OPSLOG" /><appender-ref ref="STDOUT" /></logger><logger name="historization" level="info" additivity="false"><appender-ref ref="OPSLOG" /><appender-ref ref="STDOUT" /></logger></configuration>
Restarting Rudder webapp and inventory-endpoint
[...]

Actions

Copy link

Updated by Nicolas PERRON over 11 years ago

The first display come from rudder-inventory-ldap when upgrading indexes (#2900). It is due to some commands which are not redirected to /dev/null and then appeared in the output.

The second one come from a test for the migration of the file logback.xml (#3037) which is not correctly set (it should be put in a variable).

Actions

Copy link

Updated by Nicolas PERRON over 11 years ago

Nicolas PERRON wrote:

The first display come from rudder-inventory-ldap when upgrading indexes (#2900). It is due to some commands which are not redirected to /dev/null and then appeared in the output.

For this part, the patch is harmless:

diff --git a/rudder-inventory-ldap/debian/postinst b/rudder-inventory-ldap/debian/postinst
index b15ed6e..8e812b5 100644
--- a/rudder-inventory-ldap/debian/postinst
+++ b/rudder-inventory-ldap/debian/postinst
@@ -123,7 +123,7 @@ case "$1" in
                if [ -r /opt/rudder/etc/openldap/slapd.conf -a -e /var/rudder/ldap/openldap-data/id2entry.bdb ]; then
                        grep ^index /opt/rudder/etc/openldap/slapd.conf | sed 's/\s\+/\t/g' | cut -f2 | sed 's/,/\n/g' | sort > ${SLAPD_DEFINED_INDEXES}
                        ls  /var/rudder/ldap/openldap-data/*.bdb | xargs -n 1 -I{} basename {} .bdb | sort | egrep -v '^(dn2id|id2entry)' > ${SLAPD_ACTUAL_INDEXES}
-                       if ! diff ${SLAPD_DEFINED_INDEXES} ${SLAPD_ACTUAL_INDEXES}; then
+                       if ! diff ${SLAPD_DEFINED_INDEXES} ${SLAPD_ACTUAL_INDEXES} > /dev/null; then
                                echo "OpenLDAP indexes are not up to date, reindexing..." 
                                invoke-rc.d slapd stop
                                /opt/rudder/sbin/slapindex

The second one come from a test for the migration of the file logback.xml (#3037) which is not correctly set (it should be put in a variable).

For this part, the patch is harmless too:

diff --git a/rudder-webapp/SOURCES/rudder-upgrade b/rudder-webapp/SOURCES/rudder-upgrade
index 262062d..d334505 100755
--- a/rudder-webapp/SOURCES/rudder-upgrade
+++ b/rudder-webapp/SOURCES/rudder-upgrade
@@ -561,7 +561,8 @@ if [ ${CHECK_BASEURL_ATTR} -ge 1 ]; then
 fi

 # - 2.4.0 : Update logback.xml in order to have information about name historization
-if ! cat /opt/rudder/etc/logback.xml | perl -p0e 's/\n//g' | perl -p0e 's/<!--.(?:(?!-->).)*-->//g' | perl -p0e 's/> *</></g' | grep '<logger name="historization" level="info" additivity="false"><appender-ref ref="OPSLOG" /><appen
+CHECK_LOGBACK_UPGRADE=`cat /opt/rudder/etc/logback.xml | perl -p0e 's/\n//g' | perl -p0e 's/<!--.(?:(?!-->).)*-->//g' | perl -p0e 's/> *</></g' | grep '<logger name="historization" level="info" additivity="false"><appender-ref ref
+if [ ${CHECK_LOGBACK_UPGRADE} -lt 1 ]
 then
     sed -i 's%^ *</configuration>%   <logger name="historization" level="info" additivity="false">\n     <appender-ref ref="OPSLOG" />\n     <!-- comment the following appender if you dont want to have logs about report in both st
 fi

Actions

Copy link

Updated by Nicolas PERRON over 11 years ago

This fix should be added to the 2.4.0

Actions

Copy link

Updated by Nicolas PERRON over 11 years ago

The second patch should be more like this:

diff --git a/rudder-webapp/SOURCES/rudder-upgrade b/rudder-webapp/SOURCES/rudder-upgrade
index 262062d..3fb7fd4 100755
--- a/rudder-webapp/SOURCES/rudder-upgrade
+++ b/rudder-webapp/SOURCES/rudder-upgrade
@@ -561,7 +561,7 @@ if [ ${CHECK_BASEURL_ATTR} -ge 1 ]; then
 fi

 # - 2.4.0 : Update logback.xml in order to have information about name historization
-if ! cat /opt/rudder/etc/logback.xml | perl -p0e 's/\n//g' | perl -p0e 's/<!--.(?:(?!-->).)*-->//g' | perl -p0e 's/> *</></g' | grep '<logger name="historization" level="info" additivity="false"><appender-ref ref="OPSLOG" /><appender-ref ref="STDOUT" /></logger>'
+if ! cat /opt/rudder/etc/logback.xml | perl -p0e 's/\n//g' | perl -p0e 's/<!--.(?:(?!-->).)*-->//g' | perl -p0e 's/> *</></g' | grep '<logger name="historization" level="info" additivity="false"><appender-ref ref="OPSLOG" /><appender-ref ref="STDOUT" /></logger>' > /dev/null
 then
     sed -i 's%^ *</configuration>%   <logger name="historization" level="info" additivity="false">\n     <appender-ref ref="OPSLOG" />\n     <!-- comment the following appender if you dont want to have logs about report in both stdout and opslog -->\n     <appender-ref ref="STDOUT" />\n   </logger>\n </configuration>%' /opt/rudder/etc/logback.xml
 fi

Actions

Copy link