January 2014 - Engine-commits - oVirt List Archives

Change in ovirt-dwh[master]: packaging: on upgrade first backup
by ydary＠redhat.com 02 Jan '14

02 Jan '14

Yaniv Dary has submitted this change and it was merged. Change subject: packaging: on upgrade first backup ...................................................................... packaging: on upgrade first backup Backup before doing any changes to the database Dump only schema for changing ownership Run all ownership changes in a single psql command Bug-Url: https://bugzilla.redhat.com/1046679 Bug-Url: https://bugzilla.redhat.com/1046676 Change-Id: I1b85a6aee99e94856b233f1c2285fd69e31e053f Signed-off-by: Yedidyah Bar David <didi(a)redhat.com> --- M packaging/legacy-setup/common_utils.py M packaging/legacy-setup/ovirt-engine-dwh-setup.py 2 files changed, 155 insertions(+), 81 deletions(-) Approvals: Yedidyah Bar David: Verified Yaniv Dary: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/22797 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1b85a6aee99e94856b233f1c2285fd69e31e053f Gerrit-PatchSet: 4 Gerrit-Project: ovirt-dwh Gerrit-Branch: master Gerrit-Owner: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: Ofer Schreiber <oschreib(a)redhat.com> Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com>

1 0

Change in ovirt-engine[master]: host-deploy: new registration interface
by yzaslavs＠redhat.com 02 Jan '14

02 Jan '14

Yair Zaslavsky has submitted this change and it was merged. Change subject: host-deploy: new registration interface ...................................................................... host-deploy: new registration interface Introduce structured registration interface, based on query string parameters. Sadly, we could not reach consensus for removing this legacy interface in favor of using the public supported rest api of the engine. Unlike legacy interface, new registration interface supports all services within single URI. version 0: Legacy is missing version field, version is 0. 1: Current version. VERSION 0 vds_ip Host address to register. port VDSM port within host. vds_name Host name. vds_unique_id Unique id of host. Returns: Timestamp. VERSION 1 command==get-version Get most up to date interface version. Returns: Content-Type: text-plain Content: version command==get-pki-trust Returns: Internal PEM encoded CA certificate. command==get-ssh-trust Returns: Engine ssh public key. command==register address Host address to register, default request origin. sshPort SSH port within host, default 22. sshUser SSH user to use. vdsPort VDSM port within host, default 54321. name Host name default address. uniqueId Unique id of host. Returns: 'OK' Change-Id: I54abaa8a46ac8821374e40c41d1e97a40e04ab1f Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M backend/manager/modules/services/src/main/java/org/ovirt/engine/core/services/RegisterServlet.java 1 file changed, 258 insertions(+), 78 deletions(-) Approvals: Yair Zaslavsky: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/20815 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I54abaa8a46ac8821374e40c41d1e97a40e04ab1f Gerrit-PatchSet: 8 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: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine-sdk-java[master]: codegen: Remove the hardcoded root URL
by juan.hernandez＠redhat.com 01 Jan '14

01 Jan '14

Juan Hernandez has submitted this change and it was merged. Change subject: codegen: Remove the hardcoded root URL ...................................................................... codegen: Remove the hardcoded root URL This is required in order to run the generator using a server that has been configured to use an URL different to the default. Change-Id: I77a63f8d3ad4d5fa759703be0857fac8e9b50da2 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M ovirt-engine-sdk-java-codegen/src/main/java/org/ovirt/engine/sdk/codegen/rsdl/RsdlCodegen.java 1 file changed, 10 insertions(+), 2 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/22552 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I77a63f8d3ad4d5fa759703be0857fac8e9b50da2 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine-sdk-java Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Michael Pasternak <mpastern(a)redhat.com>

1 0

Change in ovirt-engine-sdk-java[master]: sdk: Fix typo in private class name
by juan.hernandez＠redhat.com 01 Jan '14

01 Jan '14

Juan Hernandez has submitted this change and it was merged. Change subject: sdk: Fix typo in private class name ...................................................................... sdk: Fix typo in private class name This patch fixes a typo in the names of classes used internally by the header and URL builders. Change-Id: I18bbd68546f5eb7e4dc10e014901cf1e246eec14 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M ovirt-engine-sdk-java/src/main/java/org/ovirt/engine/sdk/utils/HttpHeaderBuilder.java M ovirt-engine-sdk-java/src/main/java/org/ovirt/engine/sdk/utils/UrlBuilder.java 2 files changed, 15 insertions(+), 15 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/22892 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I18bbd68546f5eb7e4dc10e014901cf1e246eec14 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-sdk-java Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com>

1 0

Change in ovirt-engine[master]: core: run HA VMs that went down right before engine stopped
by ofrenkel＠redhat.com 01 Jan '14

01 Jan '14

Omer Frenkel has submitted this change and it was merged. Change subject: core: run HA VMs that went down right before engine stopped ...................................................................... core: run HA VMs that went down right before engine stopped This patch solves a known problem that was exists before: when HA VMs went down, the engine (VdsUpdateRunTimeInfo) detected it (and updated the DB) but didn't manage to run the VM, we won't try to run the VM when the engine start. This problem is solved by fetching from the DB all the HA VMs which are down with exit status ERROR when initializing the AutoStartVmsRunner job, and those VMs are set as the first VMs the job will try to run on its first iteration. Change-Id: I2d5876f196819b2a69be0b71287c5325a8ff9dc9 Signed-off-by: Arik Hadas <ahadas(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/AutoStartVmsRunner.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/VmDAO.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/VmDAODbFacadeImpl.java M backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/VmDAOTest.java M backend/manager/modules/dal/src/test/resources/fixtures.xml M packaging/dbscripts/vms_sp.sql 6 files changed, 58 insertions(+), 6 deletions(-) Approvals: Omer Frenkel: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/19502 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2d5876f196819b2a69be0b71287c5325a8ff9dc9 Gerrit-PatchSet: 18 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: Doron Fediuck <dfediuck(a)redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(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[master]: engine: Avoid fetching null labels
by masayag＠redhat.com 01 Jan '14

01 Jan '14

Moti Asayag has submitted this change and it was merged. Change subject: engine: Avoid fetching null labels ...................................................................... engine: Avoid fetching null labels It make sense to fetch only the defined labels in the data center. Change-Id: Ib068f8a87848dfc9845733a535e05a25934082f6 Signed-off-by: Moti Asayag <masayag(a)redhat.com> --- M packaging/dbscripts/network_sp.sql 1 file changed, 2 insertions(+), 1 deletion(-) Approvals: Moti Asayag: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/22875 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib068f8a87848dfc9845733a535e05a25934082f6 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Mike Kolesnik <mkolesni(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: packaging: build: remove executable from template
by sbonazzo＠redhat.com 01 Jan '14

01 Jan '14

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: build: remove executable from template ...................................................................... packaging: build: remove executable from template Change-Id: I005fe3e04bf4d21a6596e58f2ee97249bdfe62d0 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M build/python-check.sh.in 1 file changed, 0 insertions(+), 0 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/22826 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I005fe3e04bf4d21a6596e58f2ee97249bdfe62d0 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: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: packaging: pythonlib: adopt utils procesTemplate and escape
by sbonazzo＠redhat.com 01 Jan '14

01 Jan '14

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: pythonlib: adopt utils procesTemplate and escape ...................................................................... packaging: pythonlib: adopt utils procesTemplate and escape Change-Id: I842f44ca4915e892074b75e2349ad8ab761f773d Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M packaging/pythonlib/ovirt_engine/util.py M packaging/setup/ovirt_engine_setup/util.py M packaging/setup/plugins/ovirt-engine-common/base/db/pgpass.py M packaging/setup/plugins/ovirt-engine-common/base/network/firewall_manager_iptables.py M packaging/setup/plugins/ovirt-engine-common/base/network/process_firewalld_services.py M packaging/setup/plugins/ovirt-engine-rename/core/pki.py M packaging/setup/plugins/ovirt-engine-setup/ovirt-engine/apache/ovirt_engine.py M packaging/setup/plugins/ovirt-engine-setup/ovirt-engine/config/ca.py M packaging/setup/plugins/ovirt-engine-setup/ovirt-engine/config/database.py M packaging/setup/plugins/ovirt-engine-setup/ovirt-engine/pki/ca.py 10 files changed, 55 insertions(+), 37 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/22827 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I842f44ca4915e892074b75e2349ad8ab761f773d 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: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: packaging: services: ovirt-engine: cleanup tmpdir usage
by sbonazzo＠redhat.com 01 Jan '14

01 Jan '14

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: services: ovirt-engine: cleanup tmpdir usage ...................................................................... packaging: services: ovirt-engine: cleanup tmpdir usage Change-Id: Id4af2c23ca669bea2d05a5e82f84aa20d84c9cf2 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M packaging/services/ovirt-engine/ovirt-engine.py 1 file changed, 12 insertions(+), 7 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/22828 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id4af2c23ca669bea2d05a5e82f84aa20d84c9cf2 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: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: packaging: pythonlib: service: improvements
by sbonazzo＠redhat.com 01 Jan '14

01 Jan '14

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: pythonlib: service: improvements ...................................................................... packaging: pythonlib: service: improvements 1. support with x as. 2. support random directory. 3. allow wrapped directory retrieval via property. Change-Id: I1144a58d98760168ed24a93daae225ba769f42b7 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M packaging/pythonlib/ovirt_engine/service.py 1 file changed, 9 insertions(+), 1 deletion(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/22825 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1144a58d98760168ed24a93daae225ba769f42b7 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: oVirt Jenkins CI Server

1 0