[Bugs] [Bug 1357759] Geo-rep silently ignores config parser errors

[bugzilla at redhat.com]

https://bugzilla.redhat.com/show_bug.cgi?id=1357759



--- Comment #2 from Vijay Bellur <vbellur at redhat.com> ---
COMMIT: http://review.gluster.org/14946 committed in release-3.7 by Aravinda VK
(avishwan at redhat.com) 
------
commit deaecdf4320be44ae4a8300c969f724582959067
Author: Aravinda VK <avishwan at redhat.com>
Date:   Thu Jun 23 11:53:36 2016 +0530

    geo-rep: Handle Config parser errors

    Python ConfigParser lib has two methods, readfp and read, it should
    be used as follows.

        readfp(open("defaults.conf"))
        read("custom.conf")

    ConfigParser.read(path) ignores any file errors, which is intentional
    since errors are handled while loading default config.

    Geo-rep uses only one config file(Session config in Master side and
    Template config on Slave side) so we should use readfp to avoid
    skipping OS errors.

    config.read is retained in case of `--config-set-rx` where glusterd
    creates new template config file.

    BUG: 1357759
    Change-Id: I15a14d3743facd7b8c7af0edc70fdefaa43efd04
    Signed-off-by: Aravinda VK <avishwan at redhat.com>
    Reviewed-on: http://review.gluster.org/14777
    (cherry picked from commit d94bf608b16b82f2c8f8588a96459cb746773b32)
    Reviewed-on: http://review.gluster.org/14946
    Smoke: Gluster Build System <jenkins at build.gluster.org>
    NetBSD-regression: NetBSD Build System <jenkins at build.gluster.org>
    Reviewed-by: Kotresh HR <khiremat at redhat.com>
    CentOS-regression: Gluster Build System <jenkins at build.gluster.org>

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=bZXqxmJIaD&a=cc_unsubscribe