February 2014 - Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: webadmin: New Data Center "Guide Me" does not reflect hosts ...
by emesika＠redhat.com 18 Feb '14

18 Feb '14

Eli Mesika has submitted this change and it was merged. Change subject: webadmin: New Data Center "Guide Me" does not reflect hosts presence ...................................................................... webadmin: New Data Center "Guide Me" does not reflect hosts presence DataCenter guide me dialog does not reflect the presence of hosts. "Configure Host" button appears under "There are still unconfigured entities:" even when hosts exist in the datacenter. Change-Id: I6dbbac65e439f0d58b4104da8e1acb2860e7556c Bug-Url: https://bugzilla.redhat.com/1023330 Signed-off-by: Ravi Nori <rnori(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/datacenters/DataCenterGuideModel.java M frontend/webadmin/modules/uicompat/src/main/java/org/ovirt/engine/ui/uicompat/UIConstants.java 2 files changed, 13 insertions(+), 2 deletions(-) Approvals: Eli Mesika: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/23816 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6dbbac65e439f0d58b4104da8e1acb2860e7556c Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Ravi Nori <rnori(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Einav Cohen <ecohen(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Ravi Nori <rnori(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: engine: Refactor AuditLog severity handling - part 2
by emesika＠redhat.com 18 Feb '14

18 Feb '14

Eli Mesika has submitted this change and it was merged. Change subject: engine: Refactor AuditLog severity handling - part 2 ...................................................................... engine: Refactor AuditLog severity handling - part 2 This patch removes the now unused AuditLogDirector.severities map and the associated test. Change-Id: I4dae3cf1a7d9adf8c74dbf9fa989a8e7617a7944 Signed-off-by: Martin Sivak <msivak(a)redhat.com> --- M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dal/dbbroker/auditloghandling/AuditLogDirector.java M backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dal/dbbroker/auditloghandling/AuditLogDirectorTest.java 2 files changed, 0 insertions(+), 906 deletions(-) Approvals: Martin Sivák: Verified Martin Peřina: Looks good to me, but someone else must approve Eli Mesika: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/24442 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4dae3cf1a7d9adf8c74dbf9fa989a8e7617a7944 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: tools: Removes legacy CLIParser
by emesika＠redhat.com 18 Feb '14

18 Feb '14

Eli Mesika has submitted this change and it was merged. Change subject: tools: Removes legacy CLIParser ...................................................................... tools: Removes legacy CLIParser Removes legacy class CLIParser for command line argument parsing. Change-Id: I5ba75031178d6f2b9500f932fc4d547f43066b2d Signed-off-by: Martin Perina <mperina(a)redhat.com> --- D backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/CLIParser.java 1 file changed, 0 insertions(+), 36 deletions(-) Approvals: Martin Peřina: Verified Eli Mesika: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/24665 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5ba75031178d6f2b9500f932fc4d547f43066b2d 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: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: tools: Remove dead code from KerberosConfigCheck
by emesika＠redhat.com 18 Feb '14

18 Feb '14

Eli Mesika has submitted this change and it was merged. Change subject: tools: Remove dead code from KerberosConfigCheck ...................................................................... tools: Remove dead code from KerberosConfigCheck Removes dead code (not called from project) from KerberosConfigCheck class. Change-Id: I6521dd2b5bdc5ecc46b722528196886e4ee97328 Signed-off-by: Martin Perina <mperina(a)redhat.com> --- M backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/kerberos/KerberosConfigCheck.java 1 file changed, 1 insertion(+), 46 deletions(-) Approvals: Martin Peřina: Verified Eli Mesika: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/24664 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6521dd2b5bdc5ecc46b722528196886e4ee97328 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: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: tools: Remove dead code from KrbConfCreator
by emesika＠redhat.com 18 Feb '14

18 Feb '14

Eli Mesika has submitted this change and it was merged. Change subject: tools: Remove dead code from KrbConfCreator ...................................................................... tools: Remove dead code from KrbConfCreator Removes dead code (not called from project) from KrbConfCreator class. Change-Id: I8d1fc4bac6480c40b0f6ff3df81f33570b4c0608 Signed-off-by: Martin Perina <mperina(a)redhat.com> --- M backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/kerberos/KrbConfCreator.java 1 file changed, 0 insertions(+), 65 deletions(-) Approvals: Martin Peřina: Verified Eli Mesika: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/24663 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8d1fc4bac6480c40b0f6ff3df81f33570b4c0608 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: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: core: remove redundant transaction on run vm
by ahadas＠redhat.com 18 Feb '14

18 Feb '14

Arik Hadas has submitted this change and it was merged. Change subject: core: remove redundant transaction on run vm ...................................................................... core: remove redundant transaction on run vm Saving the VM dynamic information that we get back from VDSM after calling VDSM to create a VM was used to be executed in transaction because we used to update information related to the VDS in the DB and we wanted it to be atomically. Now that we don't update anything else in the DB besides the dynamic information of the VM, doing the update in transaction with Required scope is redundant and we can drop the transaction. Change-Id: I9819a9defe421deab3badf1e4e1e62e7e3e3e92f Signed-off-by: Arik Hadas <ahadas(a)redhat.com> --- M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/CreateVmVDSCommand.java 1 file changed, 6 insertions(+), 16 deletions(-) Approvals: Roy Golan: Looks good to me, approved Arik Hadas: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/24551 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I9819a9defe421deab3badf1e4e1e62e7e3e3e92f Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Arik Hadas <ahadas(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: oVirt Jenkins CI Server

1 0

Change in ovirt-engine-sdk[sdk_3.3]: sdk: Prepare for the next release
by juan.hernandez＠redhat.com 18 Feb '14

18 Feb '14

Juan Hernandez has submitted this change and it was merged. Change subject: sdk: Prepare for the next release ...................................................................... sdk: Prepare for the next release In preparation for the next release this patch increases the version number and sets the RPM release number to 0.1. Change-Id: I5237ad1831165b5521d9ac0098f5ef3debb32de4 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M Makefile M setup.py 2 files changed, 3 insertions(+), 3 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/24672 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5237ad1831165b5521d9ac0098f5ef3debb32de4 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-sdk Gerrit-Branch: sdk_3.3 Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com>

1 0

Change in ovirt-engine-sdk[sdk_3.3]: sdk: Release 3.3.3.1
by juan.hernandez＠redhat.com 18 Feb '14

18 Feb '14

Juan Hernandez has submitted this change and it was merged. Change subject: sdk: Release 3.3.3.1 ...................................................................... sdk: Release 3.3.3.1 Change-Id: I8565be689eaf7de9bf2e069e072545e107734a9f Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M Makefile M ovirt-engine-sdk-python.spec.in 2 files changed, 5 insertions(+), 1 deletion(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/24671 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8565be689eaf7de9bf2e069e072545e107734a9f Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-sdk Gerrit-Branch: sdk_3.3 Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com>

1 0

Change in ovirt-engine[ovirt-engine-3.4]: packaging: post ovirt-engine-3.4.0_beta3 respin
by iheim＠redhat.com 18 Feb '14

18 Feb '14

Itamar Heim has submitted this change and it was merged. Change subject: packaging: post ovirt-engine-3.4.0_beta3 respin ...................................................................... packaging: post ovirt-engine-3.4.0_beta3 respin Change-Id: Ief621233da56278e812f987a8900318a4ee05442 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M ovirt-engine.spec.in M version.mak 2 files changed, 3 insertions(+), 3 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/24667 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ief621233da56278e812f987a8900318a4ee05442 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.4 Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Itamar Heim <iheim(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>

1 0

Change in ovirt-engine[ovirt-engine-3.3]: core: Fix vm_pools dynamic query
by iheim＠redhat.com 18 Feb '14

18 Feb '14

Itamar Heim has submitted this change and it was merged. Change subject: core: Fix vm_pools dynamic query ...................................................................... core: Fix vm_pools dynamic query This fix will improve the dynamic query behavior when querying from vm_pool based on primary key query. Change-Id: I1f91cf3986e6f981e66b5ffd11b478fc1e724a4a Bug-Url: https://bugzilla.redhat.com/1066392 Signed-off-by: lzelkha(a)redhat.com <lzelkha(a)redhat.com> --- M backend/manager/modules/searchbackend/src/main/java/org/ovirt/engine/core/searchbackend/SyntaxChecker.java M backend/manager/modules/searchbackend/src/test/java/org/ovirt/engine/core/searchbackend/SyntaxCheckerTest.java 2 files changed, 16 insertions(+), 1 deletion(-) Approvals: Eli Mesika: Looks good to me, approved Liran Zelkha: Verified -- To view, visit http://gerrit.ovirt.org/24633 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1f91cf3986e6f981e66b5ffd11b478fc1e724a4a Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.3 Gerrit-Owner: Liran Zelkha <lzelkha(a)redhat.com> Gerrit-Reviewer: Barak Azulay <bazulay(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: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0