March 2014 - Engine-commits - oVirt List Archives

Change in otopi[master]: packaging: post otopi-1.2.0_rc2
by Alon Bar-Lev 04 Mar '14

04 Mar '14

Alon Bar-Lev has submitted this change and it was merged. Change subject: packaging: post otopi-1.2.0_rc2 ...................................................................... packaging: post otopi-1.2.0_rc2 Change-Id: I648f152c272b0a25f42531c83b482da27e204cb9 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M ChangeLog M configure.ac 2 files changed, 5 insertions(+), 2 deletions(-) Approvals: Alon Bar-Lev: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/25354 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I648f152c272b0a25f42531c83b482da27e204cb9 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 otopi[master]: packaging: otopi-1.2.0_rc2
by Alon Bar-Lev 04 Mar '14

04 Mar '14

Alon Bar-Lev has submitted this change and it was merged. Change subject: packaging: otopi-1.2.0_rc2 ...................................................................... packaging: otopi-1.2.0_rc2 Change-Id: Ia13c77e9261873e638e3502faa4d72b8b9627a8c Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M ChangeLog M configure.ac M otopi.spec.in 3 files changed, 6 insertions(+), 3 deletions(-) Approvals: Alon Bar-Lev: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/25353 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia13c77e9261873e638e3502faa4d72b8b9627a8c 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 otopi[master]: core: log: filter keys should not assume key exists
by Alon Bar-Lev 04 Mar '14

04 Mar '14

Alon Bar-Lev has submitted this change and it was merged. Change subject: core: log: filter keys should not assume key exists ...................................................................... core: log: filter keys should not assume key exists Change-Id: I519ad18eeded66035696b8a3ad092e62e9dd9106 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M ChangeLog M src/plugins/otopi/core/log.py 2 files changed, 2 insertions(+), 1 deletion(-) Approvals: Alon Bar-Lev: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/25352 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I519ad18eeded66035696b8a3ad092e62e9dd9106 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[ovirt-engine-3.4]: packaging: setup: fix remote database typo
by iheim＠redhat.com 04 Mar '14

04 Mar '14

Itamar Heim has submitted this change and it was merged. Change subject: packaging: setup: fix remote database typo ...................................................................... packaging: setup: fix remote database typo Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1072307 Change-Id: I9b8876f5699c7abd8953ff01295111b4a983a1aa Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M packaging/setup/plugins/ovirt-engine-setup/ovirt-engine/provisioning/postgres.py 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Alon Bar-Lev: Verified Sandro Bonazzola: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/25319 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I9b8876f5699c7abd8953ff01295111b4a983a1aa Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.4 Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Itamar Heim <iheim(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[ovirt-engine-3.4]: packaging: setup: database: move connect to database library
by iheim＠redhat.com 04 Mar '14

04 Mar '14

Itamar Heim has submitted this change and it was merged. Change subject: packaging: setup: database: move connect to database library ...................................................................... packaging: setup: database: move connect to database library this will enable use of single tweaks for different driver versions. Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1067058 Change-Id: I36aa977fd6768b1ea3a96660d8473bf6ecffa277 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M packaging/setup/ovirt_engine_setup/database.py M packaging/setup/plugins/ovirt-engine-rename/core/database.py M packaging/setup/plugins/ovirt-engine-setup/ovirt-engine-common/db/connection.py 3 files changed, 76 insertions(+), 70 deletions(-) Approvals: Alon Bar-Lev: Verified Sandro Bonazzola: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/25331 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I36aa977fd6768b1ea3a96660d8473bf6ecffa277 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.4 Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Itamar Heim <iheim(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[ovirt-engine-3.3]: core: Ensure NonOperational state is saved
by iheim＠redhat.com 04 Mar '14

04 Mar '14

Itamar Heim has submitted this change and it was merged. Change subject: core: Ensure NonOperational state is saved ...................................................................... core: Ensure NonOperational state is saved A specific scenario in VdsUpdateRuntimeInfo does not save the NonOperational status to the database. This causes the host to get stuck in an unassigned status. Change-Id: I8e4c5d3dc49f48bf46609c91f13dededb43b630f Bug-Url: https://bugzilla.redhat.com/1068926 Signed-off-by: lzelkha(a)redhat.com <lzelkha(a)redhat.com> --- M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VdsUpdateRunTimeInfo.java 1 file changed, 6 insertions(+), 3 deletions(-) Approvals: Eli Mesika: Looks good to me, approved Liran Zelkha: Verified -- To view, visit http://gerrit.ovirt.org/25303 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8e4c5d3dc49f48bf46609c91f13dededb43b630f Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.3 Gerrit-Owner: Liran Zelkha <lzelkha(a)redhat.com> Gerrit-Reviewer: David Caro <dcaroest(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Itamar Heim <iheim(a)redhat.com> Gerrit-Reviewer: Liran Zelkha <lzelkha(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-reports[master]: packaging: moved some names to variables.
by ydary＠redhat.com 04 Mar '14

04 Mar '14

Yaniv Dary has submitted this change and it was merged. Change subject: packaging: moved some names to variables. ...................................................................... packaging: moved some names to variables. Change-Id: I87227051a06921248ba7574ae1d5ebeddb5fe8b0 Signed-off-by: Yaniv Dary <ydary(a)redhat.com> --- M ovirt-engine-reports.spec.in 1 file changed, 10 insertions(+), 8 deletions(-) Approvals: Yaniv Dary: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/25338 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I87227051a06921248ba7574ae1d5ebeddb5fe8b0 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-reports Gerrit-Branch: master Gerrit-Owner: Yaniv Dary <ydary(a)redhat.com> Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com>

1 0

Change in ovirt-engine[master]: packaging: setup: database: move connect to database library
by sbonazzo＠redhat.com 04 Mar '14

04 Mar '14

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: setup: database: move connect to database library ...................................................................... packaging: setup: database: move connect to database library this will enable use of single tweaks for different driver versions. Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1067058 Change-Id: I36aa977fd6768b1ea3a96660d8473bf6ecffa277 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M packaging/setup/ovirt_engine_setup/database.py M packaging/setup/plugins/ovirt-engine-rename/core/database.py M packaging/setup/plugins/ovirt-engine-setup/ovirt-engine-common/db/connection.py 3 files changed, 76 insertions(+), 70 deletions(-) Approvals: Alon Bar-Lev: Verified Sandro Bonazzola: Looks good to me, approved Yedidyah Bar David: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/25325 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I36aa977fd6768b1ea3a96660d8473bf6ecffa277 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: core: introduce RunVmCommand#RunVmFlow
by ahadas＠redhat.com 04 Mar '14

04 Mar '14

Arik Hadas has submitted this change and it was merged. Change subject: core: introduce RunVmCommand#RunVmFlow ...................................................................... core: introduce RunVmCommand#RunVmFlow RunVmCommand is a complicated command since it contains several different flows: - Resume paused VM - Resume hibernated VM - Create snapshot and then run the VM for running stateless VMs - Remove snapshot in case the VM already contains stateless snapshot - Run VM in all other cases This patch introduces an enumeration that represents the different flows. The benefits of using an enum: - It replaces local flags that indicates the different flows (mResume, isFailedStatelessSnapshot) - It makes the code more structured - It can be used to eliminate unnecessary operations in certain flows more easily. In this patch, I used it in order to prevent initialize the VM object in flows where we don't really need it - In the context of the planned enhancements for RunVmCommand, it allows us to query the flow from outside of the command Change-Id: I2a3f062323c4246b45757d3062478c0109d70708 Signed-off-by: Arik Hadas <ahadas(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RunVmCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RunVmOnceCommand.java M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/RunVmCommandTest.java 3 files changed, 200 insertions(+), 46 deletions(-) Approvals: Arik Hadas: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/22663 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2a3f062323c4246b45757d3062478c0109d70708 Gerrit-PatchSet: 23 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-scheduler-proxy[master]: Release of version 0.1.4
by msivak＠redhat.com 04 Mar '14

04 Mar '14

Martin Sivák has submitted this change and it was merged. Change subject: Release of version 0.1.4 ...................................................................... Release of version 0.1.4 Change-Id: I372d83144d422ce8dde253516a919d2af26ddbcc Signed-off-by: Martin Sivak <msivak(a)redhat.com> --- M VERSION M ovirt-scheduler-proxy.spec.in 2 files changed, 9 insertions(+), 1 deletion(-) Approvals: Martin Sivák: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/25328 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I372d83144d422ce8dde253516a919d2af26ddbcc Gerrit-PatchSet: 1 Gerrit-Project: ovirt-scheduler-proxy Gerrit-Branch: master Gerrit-Owner: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0