Change in vdsm[ovirt-3.5-gluster]: gluster: fix AttributeError in exception.py

barumuga at redhat.com barumuga at redhat.com
Wed Aug 26 10:59:24 UTC 2015


Bala.FA 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 <tjeyasin at redhat.com>
Reviewed-on: https://gerrit.ovirt.org/41530
Reviewed-by: Francesco Romani <fromani at redhat.com>
Reviewed-by: Ramesh N <rnachimu at redhat.com>
Reviewed-by: Dan Kenigsberg <danken at redhat.com>
Continuous-Integration: Dan Kenigsberg <danken at redhat.com>
Reviewed-on: https://gerrit.ovirt.org/45276
Reviewed-by: Bala.FA <barumuga at redhat.com>
Continuous-Integration: Bala.FA <barumuga at redhat.com>
---
M vdsm/gluster/exception.py
1 file changed, 17 insertions(+), 9 deletions(-)

Approvals:
  Timothy Asir: Verified
  Bala.FA: Looks good to me, approved; Passed CI tests



-- 
To view, visit https://gerrit.ovirt.org/45276
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ic4c72daf30aee8f0024b1ef9ee064a205a500620
Gerrit-PatchSet: 5
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-3.5-gluster
Gerrit-Owner: Timothy Asir <tjeyasin at redhat.com>
Gerrit-Reviewer: Bala.FA <barumuga at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: Ramesh N <rnachimu at redhat.com>
Gerrit-Reviewer: Sahina Bose <sabose at redhat.com>
Gerrit-Reviewer: Timothy Asir <tjeyasin at redhat.com>
Gerrit-Reviewer: automation at ovirt.org


More information about the vdsm-patches mailing list