[Bugs] [Bug 1145020] [SNAPSHOT] : gluster volume info should not show the value which is not set explicitly

[bugzilla at redhat.com]

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



--- Comment #5 from Anand Avati <aavati at redhat.com> ---
COMMIT: http://review.gluster.org/8793 committed in release-3.6 by Vijay Bellur
(vbellur at redhat.com) 
------
commit b98ed29cf6c1b999a025dff2269118a9198bd987
Author: Sachin Pandit <spandit at redhat.com>
Date:   Thu Jun 26 07:51:02 2014 +0530

    cli/snapshot : gluster volume info should not show the options which are
not set explicitly.

    Problem : Even though snap-max-hard-limit, snap-max-soft-limit and
    auto-delete values were not set explicitly, It was getting showed
    in the output of gluster volume info.

    Solution : Check if the value is already present in dictionary
    (That means, it is set), If value is not present then consider
    the default value,

    NOTE : This patch doesn't solve the problem where the values
    which is set globally are being displayed in gluster volume info

    Change-Id: I61445b3d2a12eb68c38a19bea53b9051ad028050
    BUG: 1145020
    Signed-off-by: Sachin Pandit <spandit at redhat.com>
    Reviewed-on: http://review.gluster.org/8191
    Tested-by: Gluster Build System <jenkins at build.gluster.com>
    Reviewed-by: Atin Mukherjee <amukherj at redhat.com>
    Reviewed-by: Avra Sengupta <asengupt at redhat.com>
    Reviewed-by: Raghavendra Bhat <raghavendra at redhat.com>
    Reviewed-by: Kaushal M <kaushal at redhat.com>
    Signed-off-by: Sachin Pandit <spandit at redhat.com>
    Reviewed-on: http://review.gluster.org/8793
    Reviewed-by: Vijaikumar Mallikarjuna <vmallika at redhat.com>

-- 
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=ElHlAGb3c8&a=cc_unsubscribe