Kanagaraj M has submitted this change and it was merged.
Change subject: engine: Set brick port and pid when returned
......................................................................
engine: Set brick port and pid when returned
When gluster process is restarted, the gluster
CLI returns the brick port as NA. This causes
the VDS xml return handler to fail as the brick
port is expected as an integer value.
Added code to set the int values only if
returned correctly.
If brick port is N/A then brick's status should
be set offline.
Change-Id: I544b0aa4705b4bcde7638f15b5b235ba10deb3b1
Bug-Url:
https://bugzilla.redhat.com/1036039
Signed-off-by: Sahina Bose <sabose(a)redhat.com>
---
M
backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/gluster/GlusterVolumeStatusReturnForXmlRpc.java
1 file changed, 12 insertions(+), 6 deletions(-)
Approvals:
Kanagaraj M: Looks good to me, approved
Sahina Bose: Verified
--
To view, visit
http://gerrit.ovirt.org/21908
To unsubscribe, visit
http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I544b0aa4705b4bcde7638f15b5b235ba10deb3b1
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Sahina Bose <sabose(a)redhat.com>
Gerrit-Reviewer: Kanagaraj M <kmayilsa(a)redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com>
Gerrit-Reviewer: Ramesh N <rnachimu(a)redhat.com>
Gerrit-Reviewer: Sahina Bose <sabose(a)redhat.com>
Gerrit-Reviewer: Shubhendu Tripathi <shtripat(a)redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server