April 2015 - Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: packaging: setup: fix for python 2.6
by sbonazzo＠redhat.com 01 Apr '15

01 Apr '15

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: setup: fix for python 2.6 ...................................................................... packaging: setup: fix for python 2.6 Fix I99de636d4da15a253f2186084902f0f133603f4d to use python 2.6 compatible code. Change-Id: Ide1164ca2a404f67b46ff906a96ec70f4519d8b1 Signed-off-by: Yedidyah Bar David <didi(a)redhat.com> --- M packaging/setup/ovirt_engine_setup/engine_common/database.py 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Sandro Bonazzola: Looks good to me, approved Barak Korren: Looks good to me, but someone else must approve oVirt Jenkins CI Server: Looks good to me, but someone else must approve Yedidyah Bar David: Verified -- To view, visit https://gerrit.ovirt.org/39427 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ide1164ca2a404f67b46ff906a96ec70f4519d8b1 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: Barak Korren <bkorren(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: core: Skip restart of host if PM is disabled for it
by emesika＠redhat.com 01 Apr '15

01 Apr '15

Eli Mesika has submitted this change and it was merged. Change subject: core: Skip restart of host if PM is disabled for it ...................................................................... core: Skip restart of host if PM is disabled for it Fixes non responding treatment flow to not execute Kdump detection and PM restart of the host if PM is disabled for the host. That bug was introduced in patch I99770873470541bc7f6c3a7338f4bcc10f5e4fb3. Change-Id: Ib5763efcb0a14abacb77f48d3912678e651bdeed Bug-Url: https://bugzilla.redhat.com/1182510 Signed-off-by: Martin Perina <mperina(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VdsNotRespondingTreatmentCommand.java 1 file changed, 4 insertions(+), 2 deletions(-) Approvals: Martin Peřina: Verified Eli Mesika: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/39407 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib5763efcb0a14abacb77f48d3912678e651bdeed Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Ori Liel <oliel(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: core: Remove FenceExecutor and unused fence return value obj...
by emesika＠redhat.com 01 Apr '15

01 Apr '15

Eli Mesika has submitted this change and it was merged. Change subject: core: Remove FenceExecutor and unused fence return value objects ...................................................................... core: Remove FenceExecutor and unused fence return value objects 1. Removes FenceExecutor 2. Removes VDSFenceReturnValue 3. Removes FenceStatusReturnValue Change-Id: If4ded059813f1472d7546e8c7c29a2538c74c945 Bug-Url: https://bugzilla.redhat.com/1182510 Signed-off-by: Martin Perina <mperina(a)redhat.com> --- D backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/FenceExecutor.java D backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/FenceExecutorTest.java D backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/FenceStatusReturnValue.java D backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/vdscommands/VDSFenceReturnValue.java M frontend/webadmin/modules/gwt-common/src/main/resources/org/ovirt/engine/core/Common.gwt.xml 5 files changed, 0 insertions(+), 797 deletions(-) Approvals: Martin Peřina: Verified Eli Mesika: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/38972 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If4ded059813f1472d7546e8c7c29a2538c74c945 Gerrit-PatchSet: 8 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Ori Liel <oliel(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <wallaroo1976(a)gmail.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: core: Replace FenceExecutor in InitVdsOnUpCommand
by emesika＠redhat.com 01 Apr '15

01 Apr '15

Eli Mesika has submitted this change and it was merged. Change subject: core: Replace FenceExecutor in InitVdsOnUpCommand ...................................................................... core: Replace FenceExecutor in InitVdsOnUpCommand Replaces FenceExecutor with HostFenceActionExecutor in IniVdsOnUpCommand class. Change-Id: I8939f51e8f42f1dc065e74e78b6e9be7011f8082 Bug-Url: https://bugzilla.redhat.com/1182510 Signed-off-by: Martin Perina <mperina(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/InitVdsOnUpCommand.java 1 file changed, 10 insertions(+), 9 deletions(-) Approvals: Ori Liel: Looks good to me, but someone else must approve Martin Peřina: Verified Eli Mesika: Verified; Looks good to me, approved Yair Zaslavsky: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/38969 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8939f51e8f42f1dc065e74e78b6e9be7011f8082 Gerrit-PatchSet: 9 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Ori Liel <oliel(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <wallaroo1976(a)gmail.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: core: Replace FenceExecutor in SshSoftFencingCommand
by emesika＠redhat.com 01 Apr '15

01 Apr '15

Eli Mesika has submitted this change and it was merged. Change subject: core: Replace FenceExecutor in SshSoftFencingCommand ...................................................................... core: Replace FenceExecutor in SshSoftFencingCommand Replaces FenceExecutor with HostFenceActionExecutor in SshSoftFencingCommand. Change-Id: Ib2654e401ec99fd934312d77d0dd2b390dead3ff Bug-Url: https://bugzilla.redhat.com/1182510 Signed-off-by: Martin Perina <mperina(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/SshSoftFencingCommand.java 1 file changed, 2 insertions(+), 1 deletion(-) Approvals: Martin Peřina: Verified Eli Mesika: Looks good to me, approved Yair Zaslavsky: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/38970 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib2654e401ec99fd934312d77d0dd2b390dead3ff Gerrit-PatchSet: 8 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Ori Liel <oliel(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <wallaroo1976(a)gmail.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: core: Replace FenceExecutor in PmHealthCheckManager
by emesika＠redhat.com 01 Apr '15

01 Apr '15

Eli Mesika has submitted this change and it was merged. Change subject: core: Replace FenceExecutor in PmHealthCheckManager ...................................................................... core: Replace FenceExecutor in PmHealthCheckManager Replaces FenceExecutor with HostFenceActionExecutor in PmHealthCheckManager. Change-Id: I5bf7cea4e3116e7c63b9f6b118379b120c077ec9 Bug-Url: https://bugzilla.redhat.com/1182510 Signed-off-by: Martin Perina <mperina(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/pm/PmHealthCheckManager.java 1 file changed, 3 insertions(+), 6 deletions(-) Approvals: Martin Peřina: Verified Eli Mesika: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/38971 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5bf7cea4e3116e7c63b9f6b118379b120c077ec9 Gerrit-PatchSet: 8 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Ori Liel <oliel(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <wallaroo1976(a)gmail.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in jenkins[master]: Splitting the build of iso from ovirt-node rpms
by tlitovsk＠redhat.com 01 Apr '15

01 Apr '15

Anatoly Litovsky has submitted this change and it was merged. Change subject: Splitting the build of iso from ovirt-node rpms ...................................................................... Splitting the build of iso from ovirt-node rpms Change-Id: Iabe34cfe78c3a1080c9890f3a8888655c887bc15 Signed-off-by: Tolik Litovsky <tlitovsk(a)redhat.com> --- M jobs/confs/shell-scripts/build-local-ovirt-node-iso.sh M jobs/confs/shell-scripts/ovirt_node_cleanup.sh M jobs/confs/yaml/templates/ovirt-node-job-templates.yaml M jobs/confs/yaml/triggers/ovirt-node.yaml 4 files changed, 37 insertions(+), 43 deletions(-) Approvals: Fabian Deutsch: Looks good to me, but someone else must approve Anatoly Litovsky: Verified; Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/39417 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iabe34cfe78c3a1080c9890f3a8888655c887bc15 Gerrit-PatchSet: 5 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Anatoly Litovsky <tlitovsk(a)redhat.com> Gerrit-Reviewer: Anatoly Litovsky <tlitovsk(a)redhat.com> Gerrit-Reviewer: Barak Korren <bkorren(a)redhat.com> Gerrit-Reviewer: David Caro <dcaroest(a)redhat.com> Gerrit-Reviewer: Fabian Deutsch <fabiand(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-live[ovirt-live-3.5]: build: ovirt-live-3.5.2_rc3
by sbonazzo＠redhat.com 01 Apr '15

01 Apr '15

Sandro Bonazzola has submitted this change and it was merged. Change subject: build: ovirt-live-3.5.2_rc3 ...................................................................... build: ovirt-live-3.5.2_rc3 Change-Id: Ie544f53346f01a1f6ec4bc0fda4d41c519621b28 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M centos/Makefile 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/39436 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie544f53346f01a1f6ec4bc0fda4d41c519621b28 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-live Gerrit-Branch: ovirt-live-3.5 Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: core: restapi: Replace FenceExecutor in GetVdsFenceStatusQuery
by emesika＠redhat.com 01 Apr '15

01 Apr '15

Eli Mesika has submitted this change and it was merged. Change subject: core: restapi: Replace FenceExecutor in GetVdsFenceStatusQuery ...................................................................... core: restapi: Replace FenceExecutor in GetVdsFenceStatusQuery Replaces FenceExecutor with HostFenceActionExecutor in GetVdsFenceStatusQuery. Change-Id: Icde949ea1f9507f28aab23d232e08ff93569b38d Bug-Url: https://bugzilla.redhat.com/1182510 Signed-off-by: Martin Perina <mperina(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/GetVdsFenceStatusQuery.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendHostResource.java M backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/BackendHostResourceTest.java 3 files changed, 38 insertions(+), 30 deletions(-) Approvals: Ori Liel: Looks good to me, but someone else must approve Martin Peřina: Verified Eli Mesika: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/38968 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Icde949ea1f9507f28aab23d232e08ff93569b38d Gerrit-PatchSet: 8 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Ori Liel <oliel(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: core: webadmin: Replace FenceExecutor in GetFenceAgentStatus...
by emesika＠redhat.com 01 Apr '15

01 Apr '15

Eli Mesika has submitted this change and it was merged. Change subject: core: webadmin: Replace FenceExecutor in GetFenceAgentStatusQuery ...................................................................... core: webadmin: Replace FenceExecutor in GetFenceAgentStatusQuery Replaces FenceExecutor with HostFenceActionExecutor in GetFenceAgentStatusQuery and refactors displaying result of agent test. Change-Id: I509729eda00098ea63396c13fd3a6619e55c8c76 Bug-Url: https://bugzilla.redhat.com/1182510 Signed-off-by: Martin Perina <mperina(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/GetFenceAgentStatusQuery.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VdsArchitectureHelper.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/queries/GetFenceAgentStatusParameters.java M frontend/webadmin/modules/gwt-common/src/main/resources/org/ovirt/engine/core/Common.gwt.xml M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/hosts/HostModel.java M frontend/webadmin/modules/uicompat/src/main/java/org/ovirt/engine/ui/uicompat/UIConstants.java M frontend/webadmin/modules/uicompat/src/main/java/org/ovirt/engine/ui/uicompat/UIMessages.java 7 files changed, 87 insertions(+), 62 deletions(-) Approvals: Martin Peřina: Verified Eli Mesika: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/38967 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I509729eda00098ea63396c13fd3a6619e55c8c76 Gerrit-PatchSet: 8 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Ori Liel <oliel(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0