Bug 615203 - corosync logging_daemon config parser code does not behave as expected
corosync logging_daemon config parser code does not behave as expected
Status: CLOSED CURRENTRELEASE
Product: Red Hat Enterprise Linux 6
Classification: Red Hat
Component: corosync (Show other bugs)
6.0
All Linux
low Severity medium
: rc
: ---
Assigned To: Fabio Massimo Di Nitto
Cluster QE
:
Depends On: 614961
Blocks: 615202
  Show dependency treegraph
 
Reported: 2010-07-16 03:05 EDT by Fabio Massimo Di Nitto
Modified: 2012-06-11 12:14 EDT (History)
12 users (show)

See Also:
Fixed In Version: corosync-1.2.3-14.el6
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: 614961
Environment:
Last Closed: 2010-11-10 17:07:24 EST
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---


Attachments (Terms of Use)
proposed patch (1.22 KB, patch)
2010-07-16 03:13 EDT, Fabio Massimo Di Nitto
no flags Details | Diff

  None (edit)
Comment 1 Fabio Massimo Di Nitto 2010-07-16 03:12:58 EDT
The logging_daemon config parser in exec/mainconfig.c I wrote several months ago contains 2 bugs.

The first problem is that logging_daemon, in the current implementation, requires a subsys: XXX entry. This is true only for corosync, as other daemons do not have subsystems.

The second problem is that in the original idea of logging configuration, from a cluster point of view:

<logging>
 <logging_daemon name="corosync" debug="on"/>
</logging>

would enable debugging (or any other option really) for all of corosync, regardless of the subsystem.

the patch in attachment addresses those 2 issues by making subsys= keyword optional (the lack of it would change the global settings as originally designed and spec´ed).

This problem affects both trunk and flatiron.
Comment 2 Fabio Massimo Di Nitto 2010-07-16 03:13:31 EDT
Created attachment 432311 [details]
proposed patch
Comment 3 Steven Dake 2010-07-16 17:48:52 EDT
Fabio,

Please post patch upstream first, although this looks fine.

Regards
-steve
Comment 4 Fabio Massimo Di Nitto 2010-07-17 00:21:12 EDT
(In reply to comment #3)
> Fabio,
> 
> Please post patch upstream first, although this looks fine.

yeah I did both at the same time.
Comment 5 Fabio Massimo Di Nitto 2010-07-19 02:37:01 EDT
Committed revision 2997.
Comment 7 Dean Jansa 2010-08-26 18:01:06 EDT
[root@marathon-01 ~]# grep log /etc/cluster/cluster.conf
        <dlm log_debug="1"/>
        <logging>
                <logging_daemon name="corosync" debug="on"/>
        </logging>


Cluster starts, no parse errors/warnings in messages.
Comment 8 releng-rhel@redhat.com 2010-11-10 17:07:24 EST
Red Hat Enterprise Linux 6.0 is now available and should resolve
the problem described in this bug report. This report is therefore being closed
with a resolution of CURRENTRELEASE. You may reopen this bug report if the
solution does not work for you.
Comment 9 Steven Dake 2012-06-11 12:14:22 EDT
Comment on attachment 432311 [details]
proposed patch

removing review flag as patch is committed.

Note You need to log in before you can comment on or make changes to this bug.