Dan Kenigsberg has submitted this change and it was merged. Change subject: gluster: fix AttributeError in exception.py ......................................................................
gluster: fix AttributeError in exception.py Currently whenever an exception raised with extra arguments apart from rc, out, err; the exception failed with AttributeError due to base class member variables are not initialized properly. This patch fixes this AttributeError issue by calling the ancestor's __init__ method explicitly to invoke base class constructor which will initialize the base class member variables like rc, out and err. Change-Id: Ic4c72daf30aee8f0024b1ef9ee064a205a500620 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1231722 Signed-off-by: Timothy Asir Jeyasingh <[email protected]> Reviewed-on: https://gerrit.ovirt.org/41530 Reviewed-by: Francesco Romani <[email protected]> Reviewed-by: Ramesh N <[email protected]> Reviewed-by: Dan Kenigsberg <[email protected]> Continuous-Integration: Dan Kenigsberg <[email protected]> --- M vdsm/gluster/exception.py 1 file changed, 17 insertions(+), 9 deletions(-) Approvals: Timothy Asir: Verified Dan Kenigsberg: Looks good to me, approved; Passed CI tests Francesco Romani: Looks good to me, but someone else must approve Ramesh N: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/41530 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic4c72daf30aee8f0024b1ef9ee064a205a500620 Gerrit-PatchSet: 6 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Timothy Asir <[email protected]> Gerrit-Reviewer: Bala.FA <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Darshan N <[email protected]> Gerrit-Reviewer: Francesco Romani <[email protected]> Gerrit-Reviewer: Ramesh N <[email protected]> Gerrit-Reviewer: Shubhendu Tripathi <[email protected]> Gerrit-Reviewer: Timothy Asir <[email protected]> Gerrit-Reviewer: [email protected] Gerrit-Reviewer: rebecca salomi <[email protected]> _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
