Change in ovirt-engine[master]: engine: Set brick port and pid when returned

[kmayilsa at redhat.com]

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 at 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 at redhat.com>
Gerrit-Reviewer: Kanagaraj M <kmayilsa at redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofrenkel at redhat.com>
Gerrit-Reviewer: Ramesh N <rnachimu at redhat.com>
Gerrit-Reviewer: Sahina Bose <sabose at redhat.com>
Gerrit-Reviewer: Shubhendu Tripathi <shtripat at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server