- Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: webadmin : gluster volume rebalance status error handling
by kmayilsa＠redhat.com 29 Oct '13

29 Oct '13

Kanagaraj M has submitted this change and it was merged. Change subject: webadmin : gluster volume rebalance status error handling ...................................................................... webadmin : gluster volume rebalance status error handling Errors like, 1. If some/all the hosts forming a distribute volume with atleast 2 bricks are down, and if rebalance status is triggered on such a volume, approprite error message is displayed. 2. If any n/w related or any other exceptions are thrown,they are also handled properly. And hence the popup doesn't hang anymore. Change-Id: Ie13748716f23a797fc0e3ec16acbb4150c43b7e8 Signed-off-by: Anmol Babu <anbabu(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/dataprovider/AsyncDataProvider.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/volumes/VolumeListModel.java M frontend/webadmin/modules/uicompat/src/main/java/org/ovirt/engine/ui/uicompat/UIMessages.java 3 files changed, 9 insertions(+), 13 deletions(-) Approvals: anmolbabu: Verified Kanagaraj M: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/19598 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie13748716f23a797fc0e3ec16acbb4150c43b7e8 Gerrit-PatchSet: 18 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: anmolbabu <anbabu(a)redhat.com> Gerrit-Reviewer: Kanagaraj M <kmayilsa(a)redhat.com> Gerrit-Reviewer: Ramesh N <rnachimu(a)redhat.com> Gerrit-Reviewer: Shubhendu Tripathi <shtripat(a)redhat.com> Gerrit-Reviewer: anmolbabu <anbabu(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[ovirt-engine-3.3]: restapi: fix broken rsdl
by iheim＠redhat.com 29 Oct '13

29 Oct '13

Itamar Heim has submitted this change and it was merged. Change subject: restapi: fix broken rsdl ...................................................................... restapi: fix broken rsdl missing escape sequence at type definition Change-Id: Idf3bce6b705f5a5cbe3bc3ba0c1b3d1472531845 Signed-off-by: Michael pasternak <mpastern(a)redhat.com> --- M backend/manager/modules/restapi/interface/definition/src/main/resources/rsdl_metadata.yaml 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Itamar Heim: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/20645 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Idf3bce6b705f5a5cbe3bc3ba0c1b3d1472531845 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.3 Gerrit-Owner: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Itamar Heim <iheim(a)redhat.com> Gerrit-Reviewer: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: restapi: fix broken rsdl
by mpastern＠redhat.com 29 Oct '13

29 Oct '13

Michael Pasternak has submitted this change and it was merged. Change subject: restapi: fix broken rsdl ...................................................................... restapi: fix broken rsdl missing escape sequence at type definition Change-Id: Idf3bce6b705f5a5cbe3bc3ba0c1b3d1472531845 Signed-off-by: Michael pasternak <mpastern(a)redhat.com> --- M backend/manager/modules/restapi/interface/definition/src/main/resources/rsdl_metadata.yaml 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Michael Pasternak: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/20644 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Idf3bce6b705f5a5cbe3bc3ba0c1b3d1472531845 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Michael Pasternak <mpastern(a)redhat.com>

1 0

Change in ovirt-engine[ovirt-engine-3.3]: tools: Fixes SMTP configuration in notifier
by iheim＠redhat.com 29 Oct '13

29 Oct '13

Itamar Heim has submitted this change and it was merged. Change subject: tools: Fixes SMTP configuration in notifier ...................................................................... tools: Fixes SMTP configuration in notifier Fixes proper settings of JavaMail properties to support SMTPS protocol. Change-Id: If85e2d3cac58ed70e2bfa52de31c2600c2924fdd Bug-Url: https://bugzilla.redhat.com/1020900 Signed-off-by: Martin Perina <mperina(a)redhat.com> --- M backend/manager/tools/src/main/java/org/ovirt/engine/core/notifier/utils/sender/mail/JavaMailSender.java 1 file changed, 7 insertions(+), 18 deletions(-) Approvals: Martin Peřina: Verified Eli Mesika: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/20640 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If85e2d3cac58ed70e2bfa52de31c2600c2924fdd Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.3 Gerrit-Owner: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Itamar Heim <iheim(a)redhat.com> Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: webadmin : Time is shown with seconds in rebalance and remov...
by kmayilsa＠redhat.com 29 Oct '13

29 Oct '13

Kanagaraj M has submitted this change and it was merged. Change subject: webadmin : Time is shown with seconds in rebalance and remove-bricks status ...................................................................... webadmin : Time is shown with seconds in rebalance and remove-bricks status Time is shown with seconds in rebalance and remove-bricks status Change-Id: I4226646440ff721ad09e9a721441159281ffd516 Signed-off-by: Anmol Babu <anbabu(a)redhat.com> --- A frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/renderer/GlusterRebalanceDateTimeRenderer.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/gluster/VolumeRebalanceStatusPopupView.java 2 files changed, 34 insertions(+), 2 deletions(-) Approvals: Kanagaraj M: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/20613 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4226646440ff721ad09e9a721441159281ffd516 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: anmolbabu <anbabu(a)redhat.com> Gerrit-Reviewer: Kanagaraj M <kmayilsa(a)redhat.com> Gerrit-Reviewer: Ramesh N <rnachimu(a)redhat.com> Gerrit-Reviewer: Shubhendu Tripathi <shtripat(a)redhat.com> Gerrit-Reviewer: anmolbabu <anbabu(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: webadmin: updated remove-brick/rebalnce status icons
by kmayilsa＠redhat.com 29 Oct '13

29 Oct '13

Kanagaraj M has submitted this change and it was merged. Change subject: webadmin: updated remove-brick/rebalnce status icons ...................................................................... webadmin: updated remove-brick/rebalnce status icons Replaced old rebalance stoped, remove-brick stopped and failed icons with newer ones. Change-Id: I20771660cabfb1b5909e0dceea0864911bdba6fd Signed-off-by: Kanagaraj M <kmayilsa(a)redhat.com> --- M frontend/webadmin/modules/webadmin/src/main/resources/org/ovirt/engine/ui/webadmin/images/glusterAsyncTasks/rebalance_stopped.png M frontend/webadmin/modules/webadmin/src/main/resources/org/ovirt/engine/ui/webadmin/images/glusterAsyncTasks/remove_brick_failed.png M frontend/webadmin/modules/webadmin/src/main/resources/org/ovirt/engine/ui/webadmin/images/glusterAsyncTasks/remove_brick_stopped.png 3 files changed, 0 insertions(+), 0 deletions(-) Approvals: Kanagaraj M: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/20377 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I20771660cabfb1b5909e0dceea0864911bdba6fd Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Kanagaraj M <kmayilsa(a)redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Kanagaraj M <kmayilsa(a)redhat.com> Gerrit-Reviewer: Sahina Bose <sabose(a)redhat.com>

1 0

Change in ovirt-engine[master]: engine: Added hostId to task status return
by kmayilsa＠redhat.com 29 Oct '13

29 Oct '13

Kanagaraj M has submitted this change and it was merged. Change subject: engine: Added hostId to task status return ...................................................................... engine: Added hostId to task status return Added hostId to task status return as this is needed to build the host href links in REST API Change-Id: I89beb74dc5b7b8aa73ee99d811540d21bbe3eb3f Signed-off-by: Sahina Bose <sabose(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/GetGlusterVolumeRebalanceStatusQuery.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/GetGlusterVolumeRemoveBricksStatusQuery.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/gluster/GlusterVolumeTaskStatusForHost.java 3 files changed, 13 insertions(+), 0 deletions(-) Approvals: Kanagaraj M: Looks good to me, approved Sahina Bose: Verified -- To view, visit http://gerrit.ovirt.org/20403 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I89beb74dc5b7b8aa73ee99d811540d21bbe3eb3f Gerrit-PatchSet: 3 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

1 0

Change in ovirt-engine[master]: gluster:Datatype change for bytesRead & bytesWritten from in...
by kmayilsa＠redhat.com 29 Oct '13

29 Oct '13

Kanagaraj M has submitted this change and it was merged. Change subject: gluster:Datatype change for bytesRead & bytesWritten from int to long. ...................................................................... gluster:Datatype change for bytesRead & bytesWritten from int to long. This patch changes the datatype of bytesRead and bytesWritten from int to long to avoid the data overflow that may occur. Change-Id: I5dd8f372059a4d0616bef34325e23d88a959bcba Bug-url: https://bugzilla.redhat.com/show_bug.cgi?id=1015013 Signed-off-by: ndarshan <dnarayan(a)redhat.com> --- M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/gluster/GlusterClientInfo.java M backend/manager/modules/restapi/interface/definition/src/main/resources/api.xsd M backend/manager/modules/restapi/types/src/test/java/org/ovirt/engine/api/restapi/types/GlusterBrickDetailMapperTest.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/gluster/GlusterVolumeStatusReturnForXmlRpc.java 4 files changed, 12 insertions(+), 12 deletions(-) Approvals: Kanagaraj M: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/20436 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5dd8f372059a4d0616bef34325e23d88a959bcba Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Darshan N <dnarayan(a)redhat.com> Gerrit-Reviewer: Darshan N <dnarayan(a)redhat.com> Gerrit-Reviewer: Kanagaraj M <kmayilsa(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(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

1 0

Change in ovirt-engine[master]: webadmin : Fix for StopRebalanceButton not disabled on rebal...
by kmayilsa＠redhat.com 29 Oct '13

29 Oct '13

Kanagaraj M has submitted this change and it was merged. Change subject: webadmin : Fix for StopRebalanceButton not disabled on rebalance complete. ...................................................................... webadmin : Fix for StopRebalanceButton not disabled on rebalance complete. Once rebalance is completed in all the nodes and the completed text gets appended to the status at field, stop rebalance is disabled. Fix for stop rebalance from status pop up not enabled. Change-Id: I237be3005e064c3ed414f04f544909b115321594 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1020254 Signed-off-by: Anmol Babu <anbabu(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/gluster/VolumeRebalanceStatusModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/volumes/VolumeListModel.java 2 files changed, 16 insertions(+), 7 deletions(-) Approvals: Kanagaraj M: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/20422 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I237be3005e064c3ed414f04f544909b115321594 Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: anmolbabu <anbabu(a)redhat.com> Gerrit-Reviewer: Kanagaraj M <kmayilsa(a)redhat.com> Gerrit-Reviewer: anmolbabu <anbabu(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: tools: Fixes SMTP configuration in notifier
by emesika＠redhat.com 29 Oct '13

29 Oct '13

Eli Mesika has submitted this change and it was merged. Change subject: tools: Fixes SMTP configuration in notifier ...................................................................... tools: Fixes SMTP configuration in notifier Fixes proper settings of JavaMail properties to support SMTPS protocol. Change-Id: I9d5d898549f43c18afbc568413f7b9a71b4f5a28 Bug-Url: https://bugzilla.redhat.com/1020900 Signed-off-by: Martin Perina <mperina(a)redhat.com> --- M backend/manager/tools/src/main/java/org/ovirt/engine/core/notifier/utils/sender/mail/JavaMailSender.java 1 file changed, 7 insertions(+), 18 deletions(-) Approvals: Eli Mesika: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/20427 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I9d5d898549f43c18afbc568413f7b9a71b4f5a28 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Eli Mesika <elimesika(a)gmail.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: mooli tayer <mtayer(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0