October 2013 - Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: host-deploy: cleanup: rename Port->VdsPort to distinguish ss...
by emesika＠redhat.com 29 Oct '13

29 Oct '13

Eli Mesika has submitted this change and it was merged. Change subject: host-deploy: cleanup: rename Port->VdsPort to distinguish ssh and other ports ...................................................................... host-deploy: cleanup: rename Port->VdsPort to distinguish ssh and other ports Change-Id: Id2ceb930c534289f7e81d6c968e2fdfe9810022d Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RegisterVdsQuery.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/queries/RegisterVdsParameters.java 2 files changed, 15 insertions(+), 15 deletions(-) Approvals: Eli Mesika: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/20598 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id2ceb930c534289f7e81d6c968e2fdfe9810022d Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: host-deploy: cleanup: remove depreciated otp usage
by emesika＠redhat.com 29 Oct '13

29 Oct '13

Eli Mesika has submitted this change and it was merged. Change subject: host-deploy: cleanup: remove depreciated otp usage ...................................................................... host-deploy: cleanup: remove depreciated otp usage Change-Id: I68287641c891228ea1250feaa336359bc46eb7e0 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RegisterVdsQuery.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/queries/RegisterVdsParameters.java M backend/manager/modules/root/src/main/java/org/ovirt/engine/core/register/RegisterServlet.java 3 files changed, 3 insertions(+), 53 deletions(-) Approvals: Eli Mesika: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/20597 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I68287641c891228ea1250feaa336359bc46eb7e0 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: host-deploy: cleanup: remove depreciated netmask parameter
by emesika＠redhat.com 29 Oct '13

29 Oct '13

Eli Mesika has submitted this change and it was merged. Change subject: host-deploy: cleanup: remove depreciated netmask parameter ...................................................................... host-deploy: cleanup: remove depreciated netmask parameter Change-Id: If6b32d02a2a5333c98783993d24ae683c8c1bd96 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/queries/RegisterVdsParameters.java M backend/manager/modules/root/src/main/java/org/ovirt/engine/core/register/RegisterServlet.java 2 files changed, 2 insertions(+), 16 deletions(-) Approvals: Eli Mesika: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/20596 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If6b32d02a2a5333c98783993d24ae683c8c1bd96 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-hosted-engine-setup[ovirt-hosted-engine-setup-1.0]: sanlock: support for Gluster domains
by sbonazzo＠redhat.com 29 Oct '13

29 Oct '13

Sandro Bonazzola has submitted this change and it was merged. Change subject: sanlock: support for Gluster domains ...................................................................... sanlock: support for Gluster domains sanlock lease creation and domain monitoring didn't work on domains using glusterfs. Now the right paths for glusterfs domains are used. Change-Id: I723604e67906abfe2c7618821911cdf43f133a58 Bug-Url: https://bugzilla.redhat.com/1023962 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M src/plugins/ovirt-hosted-engine-setup/sanlock/lockspace.py M src/plugins/ovirt-hosted-engine-setup/storage/storage.py 2 files changed, 18 insertions(+), 6 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/20650 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I723604e67906abfe2c7618821911cdf43f133a58 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-hosted-engine-setup Gerrit-Branch: ovirt-hosted-engine-setup-1.0 Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-hosted-engine-setup[master]: sanlock: support for Gluster domains
by sbonazzo＠redhat.com 29 Oct '13

29 Oct '13

Sandro Bonazzola has submitted this change and it was merged. Change subject: sanlock: support for Gluster domains ...................................................................... sanlock: support for Gluster domains sanlock lease creation and domain monitoring didn't work on domains using glusterfs. Now the right paths for glusterfs domains are used. Change-Id: I723604e67906abfe2c7618821911cdf43f133a58 Bug-Url: https://bugzilla.redhat.com/1023962 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M src/plugins/ovirt-hosted-engine-setup/sanlock/lockspace.py M src/plugins/ovirt-hosted-engine-setup/storage/storage.py 2 files changed, 18 insertions(+), 6 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Greg Padgett: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/20617 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I723604e67906abfe2c7618821911cdf43f133a58 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-hosted-engine-setup Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Alex Lourie <alourie(a)redhat.com> Gerrit-Reviewer: Greg Padgett <gpadgett(a)redhat.com> Gerrit-Reviewer: Leonid Natapov <leonidnatapov(a)gmail.com> Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

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