July 2013 - Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: engine: fix 2 findbugs errors
by gchaplik＠redhat.com 17 Jul '13

17 Jul '13

Gilad Chaplik has submitted this change and it was merged. Change subject: engine: fix 2 findbugs errors ...................................................................... engine: fix 2 findbugs errors Mistakenly merged while introducing cluster policy feature: 1) new unused variable in ClusterModel. 2) referencing an object prior a nullity check. Signed-off-by: Gilad Chaplik <gchaplik(a)redhat.com> Change-Id: I7c83849a810e398b70c2488ab8a9fd2550e4ce17 --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VmPoolCommandBase.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/clusters/ClusterModel.java 2 files changed, 1 insertion(+), 2 deletions(-) Approvals: Laszlo Hornyak: Verified; Looks good to me, approved Gilad Chaplik: Verified -- To view, visit http://gerrit.ovirt.org/16984 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I7c83849a810e398b70c2488ab8a9fd2550e4ce17 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Laszlo Hornyak <lhornyak(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-log-collector[master]: ssh: avoid connection timeouts
by sbonazzo＠redhat.com 17 Jul '13

17 Jul '13

Sandro Bonazzola has submitted this change and it was merged. Change subject: ssh: avoid connection timeouts ...................................................................... ssh: avoid connection timeouts On some hypervistor the sosreport collection can take more than 900 seconds (default ClientAliveInterval). Calling ssh with -oServerAliveInterval=600 avoid the connection timeout for ServerAliveCountMax (deafults=3) times. Change-Id: I19075dd1a39ed0f9f7242897157d017c2375aad6 Bug-Url: https://bugzilla.redhat.com/962677 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M src/__main__.py 1 file changed, 4 insertions(+), 0 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/16293 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I19075dd1a39ed0f9f7242897157d017c2375aad6 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-log-collector Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Keith Robertson <kroberts(a)redhat.com> Gerrit-Reviewer: Kiril Nesenko <knesenko(a)redhat.com> Gerrit-Reviewer: Moran Goldboim <mgoldboi(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com>

1 0

Change in ovirt-engine[master]: tools: Add stacktrace debugging to DnsSrvLocator
by yzaslavs＠redhat.com 17 Jul '13

17 Jul '13

Yair Zaslavsky has submitted this change and it was merged. Change subject: tools: Add stacktrace debugging to DnsSrvLocator ...................................................................... tools: Add stacktrace debugging to DnsSrvLocator Change-Id: Ibd0ab8a31dc26b96b5f8731dfe288642a1a0a91b Signed-off-by: Yair Zaslavsky <yzaslavs(a)redhat.com> --- M backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/dns/DnsSRVLocator.java 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: Yair Zaslavsky: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/16980 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ibd0ab8a31dc26b96b5f8731dfe288642a1a0a91b Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: gluster: Corrected duplicate entries in audit logs
by kmayilsa＠redhat.com 17 Jul '13

17 Jul '13

Kanagaraj M has submitted this change and it was merged. Change subject: gluster: Corrected duplicate entries in audit logs ...................................................................... gluster: Corrected duplicate entries in audit logs Removed duplicate entries from AuditLogMessages.properties Change-Id: I3822193bbfe5bb84c0d7e077f394d5d12dea0e08 Signed-off-by: Shubhendu Tripathi <shtripat(a)redhat.com> --- M backend/manager/modules/dal/src/main/resources/bundles/AuditLogMessages.properties 1 file changed, 0 insertions(+), 3 deletions(-) Approvals: Shubhendu Tripathi: Verified Kanagaraj M: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/16896 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I3822193bbfe5bb84c0d7e077f394d5d12dea0e08 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Shubhendu Tripathi <shtripat(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: Tal Nisan <tnisan(a)redhat.com>

1 0

Change in ovirt-host-deploy[master]: vdsm: packages: use new method of libvirt reconfigure
by Alon Bar-Lev 16 Jul '13

16 Jul '13

Alon Bar-Lev has submitted this change and it was merged. Change subject: vdsm: packages: use new method of libvirt reconfigure ...................................................................... vdsm: packages: use new method of libvirt reconfigure Change-Id: Ib8b29fb7b8d9e44e55c4a44a25378234d39afb54 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M ChangeLog M src/plugins/ovirt-host-deploy/vdsm/packages.py 2 files changed, 25 insertions(+), 14 deletions(-) Approvals: Alon Bar-Lev: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/16962 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib8b29fb7b8d9e44e55c4a44a25378234d39afb54 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-host-deploy Gerrit-Branch: master Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com>

1 0

Change in otopi[master]: network: ssh: make sure directory owner is the user
by Alon Bar-Lev 16 Jul '13

16 Jul '13

Alon Bar-Lev has submitted this change and it was merged. Change subject: network: ssh: make sure directory owner is the user ...................................................................... network: ssh: make sure directory owner is the user Change-Id: I598856e7a320131dde9f4f6d68b8fe171014a3fa Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M src/plugins/otopi/network/ssh.py 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: Alon Bar-Lev: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/16957 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I598856e7a320131dde9f4f6d68b8fe171014a3fa Gerrit-PatchSet: 1 Gerrit-Project: otopi Gerrit-Branch: master Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com>

1 0

Change in ovirt-engine[master]: webadmin: add feedback image to webadmin
by derez＠redhat.com 16 Jul '13

16 Jul '13

Daniel Erez has submitted this change and it was merged. Change subject: webadmin: add feedback image to webadmin ...................................................................... webadmin: add feedback image to webadmin Signed-off-by: Einav Cohen <ecohen(a)redhat.com> Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> Signed-off-by: Gilad Chaplik <gchaplik(a)redhat.com> Change-Id: Ic804e1ec2a7365898db7013a09639a2096a76184 --- M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/utils/DynamicMessages.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/ApplicationConstants.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/ApplicationDynamicMessages.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/presenter/HeaderPresenterWidget.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/HeaderView.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/HeaderView.ui.xml A packaging/branding/ovirt.brand/images/feedback_bg.png A packaging/branding/ovirt.brand/images/feedback_left.png M packaging/branding/ovirt.brand/messages.properties M packaging/branding/ovirt.brand/web_admin.css 10 files changed, 97 insertions(+), 5 deletions(-) Approvals: Daniel Erez: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/15984 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic804e1ec2a7365898db7013a09639a2096a76184 Gerrit-PatchSet: 10 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Einav Cohen <ecohen(a)redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Greg Sheremeta <gshereme(a)redhat.com> Gerrit-Reviewer: Itamar Heim <iheim(a)redhat.com> Gerrit-Reviewer: Vojtech Szocs <vszocs(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: engine: Adding new VDSM error codes and messages
by amureini＠redhat.com 16 Jul '13

16 Jul '13

Allon Mureinik has submitted this change and it was merged. Change subject: engine: Adding new VDSM error codes and messages ...................................................................... engine: Adding new VDSM error codes and messages Adding new error code VolumeResizeValueError(232) and messages for already existing errors ResizeErr(58) and UpdateDevice(56). Change-Id: I5e0edf25fa07e4c365ba9ad841a3bf389fe16a60 Signed-off-by: Sergey Gotliv <sgotliv(a)redhat.com> --- M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/errors/VdcBllErrors.java M backend/manager/modules/dal/src/main/resources/bundles/VdsmErrors.properties 2 files changed, 5 insertions(+), 1 deletion(-) Approvals: Allon Mureinik: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/16914 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5e0edf25fa07e4c365ba9ad841a3bf389fe16a60 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Sergey Gotliv <sgotliv(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine-cli[master]: sdk: prepare for the next development iteration
by mpastern＠redhat.com 16 Jul '13

16 Jul '13

Michael Pasternak has submitted this change and it was merged. Change subject: sdk: prepare for the next development iteration ...................................................................... sdk: prepare for the next development iteration Change-Id: I98a718b30354f5a439e74328f34463c553e12f98 Signed-off-by: Michael pasternak <mpastern(a)redhat.com> --- M Makefile M setup.py 2 files changed, 3 insertions(+), 3 deletions(-) Approvals: Michael Pasternak: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/16947 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I98a718b30354f5a439e74328f34463c553e12f98 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine-cli 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-sdk[master]: sdk: prepare for the next development iteration
by mpastern＠redhat.com 16 Jul '13

16 Jul '13

Michael Pasternak has submitted this change and it was merged. Change subject: sdk: prepare for the next development iteration ...................................................................... sdk: prepare for the next development iteration Change-Id: I5f56778f4bf675c074acacefa9158f6c21726814 Signed-off-by: Michael pasternak <mpastern(a)redhat.com> --- M Makefile M setup.py 2 files changed, 3 insertions(+), 3 deletions(-) Approvals: Michael Pasternak: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/16946 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5f56778f4bf675c074acacefa9158f6c21726814 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine-sdk Gerrit-Branch: master Gerrit-Owner: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Michael Pasternak <mpastern(a)redhat.com>

1 0