August 2015 - Engine-commits - oVirt List Archives

Change in jenkins[master]: Adding ovirt-image-uploader standard jobs
by sbonazzo＠redhat.com 04 Aug '15

04 Aug '15

Sandro Bonazzola has submitted this change and it was merged. Change subject: Adding ovirt-image-uploader standard jobs ...................................................................... Adding ovirt-image-uploader standard jobs Including adding them to the publisher Change-Id: I1b2dffc4d3c52f609ac9707b6030a3f893f7c488 Signed-off-by: David Caro <dcaroest(a)redhat.com> --- D jobs/confs/yaml/jobs/ovirt-image-uploader/ovirt-image-uploader_create-rpms.yaml A jobs/confs/yaml/jobs/ovirt-image-uploader/ovirt-image-uploader_standard.yaml M jobs/confs/yaml/jobs/ovirt/publish-rpms-nightly-master.yaml 3 files changed, 40 insertions(+), 48 deletions(-) Approvals: Sandro Bonazzola: Looks good to me; Ready for merge Jenkins CI: Passed CI tests David Caro: Verified; Ready for review -- To view, visit https://gerrit.ovirt.org/44231 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1b2dffc4d3c52f609ac9707b6030a3f893f7c488 Gerrit-PatchSet: 4 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: David Caro <dcaroest(a)redhat.com> Gerrit-Reviewer: David Caro <dcaroest(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>

1 0

Change in ovirt-image-uploader[ovirt-image-uploader-3.5]: Added standard ci scripts
by sbonazzo＠redhat.com 04 Aug '15

04 Aug '15

Sandro Bonazzola has submitted this change and it was merged. Change subject: Added standard ci scripts ...................................................................... Added standard ci scripts This way it can run completely inside mock Change-Id: Ia812dcd02d49453f05315f3727aa006183346153 Signed-off-by: David Caro <dcaroest(a)redhat.com> (cherry picked from commit 8d50380f5178b31a729fd6f5f8c1e01cd3b8c3a9) Signed-off-by: David Caro <dcaroest(a)redhat.com> --- M .gitignore A automation/build-artifacts.packages A automation/build-artifacts.repos A automation/build-artifacts.sh A automation/check-patch.packages A automation/check-patch.sh 6 files changed, 34 insertions(+), 0 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, approved Jenkins CI: Passed CI tests David Caro: Verified -- To view, visit https://gerrit.ovirt.org/44233 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia812dcd02d49453f05315f3727aa006183346153 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-image-uploader Gerrit-Branch: ovirt-image-uploader-3.5 Gerrit-Owner: David Caro <dcaroest(a)redhat.com> Gerrit-Reviewer: Barak Korren <bkorren(a)redhat.com> Gerrit-Reviewer: David Caro <dcaroest(a)redhat.com> Gerrit-Reviewer: Doron Fediuck <dfediuck(a)redhat.com> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Max Kovgan <mvk(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Sagi Shnaidman <sshnaidm(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: VM pinning to multiple hosts (dwh fix)
by rgolan＠redhat.com 04 Aug '15

04 Aug '15

Roy Golan has submitted this change and it was merged. Change subject: core: VM pinning to multiple hosts (dwh fix) ...................................................................... core: VM pinning to multiple hosts (dwh fix) The initial feature breaks the DWH database view. Failing DWH install. Fix dwh_vm_configuration_history_view in create_dwh_views.sql To populate single pinned host uuid into default_host field. This is a temporary fix, till DWH team will be able to attend this feature. The DWH will reflect only one pinned host. Not nessearily the current or last run. In simple case when there is only single pinned host or none, it will be correct. Thanks to szacks(a)redhat.com who helped with 'SELECT DISTINCT ON' databae feature. Change-Id: I5ffa7eb9b214f288c96993cc553e4511b5bc1944 Bug-Url: https://bugzilla.redhat.com/1107512 Signed-off-by: Dudi Maroshi <dudi(a)redhat.com> --- M packaging/dbscripts/create_dwh_views.sql 1 file changed, 4 insertions(+), 1 deletion(-) Approvals: Dudi Maroshi: Verified Eli Mesika: Looks good to me, but someone else must approve Jenkins CI: Passed CI tests Roy Golan: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/43864 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5ffa7eb9b214f288c96993cc553e4511b5bc1944 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Dudi Maroshi <dudi(a)redhat.com> Gerrit-Reviewer: Dudi Maroshi <dudi(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Sim Zacks <szacks(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: engine: Apparently, potgres 9 doesn't support 'RETURNS TABLE'
by alkaplan＠redhat.com 04 Aug '15

04 Aug '15

Alona Kaplan has submitted this change and it was merged. Change subject: engine: Apparently, potgres 9 doesn't support 'RETURNS TABLE' ...................................................................... engine: Apparently, potgres 9 doesn't support 'RETURNS TABLE' Changed the only stored procedure that used it to resturn 'setof' instead. Change-Id: I5628f6e5dfccc10e12e7f9a5db879724704830f9 Bug-Url: https://bugzilla.redhat.com/1249671 Signed-off-by: Alona Kaplan <alkaplan(a)redhat.com> --- M packaging/dbscripts/network_sp.sql 1 file changed, 10 insertions(+), 1 deletion(-) Approvals: Alona Kaplan: Verified; Passed CI tests Oved Ourfali: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/44337 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5628f6e5dfccc10e12e7f9a5db879724704830f9 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Alona Kaplan <alkaplan(a)redhat.com> Gerrit-Reviewer: Alona Kaplan <alkaplan(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: core: import vm from virtual appliance
by ahadas＠redhat.com 04 Aug '15

04 Aug '15

Arik Hadas has submitted this change and it was merged. Change subject: core: import vm from virtual appliance ...................................................................... core: import vm from virtual appliance Introduce import-VM command for importing a VM from OVA file. Change-Id: I0f76282ffb9c9a6882763dcb87259e479bfc4bf5 Signed-off-by: Arik Hadas <ahadas(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/AddDiskCommand.java A backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ConvertOvaCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ConvertVmCallback.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ConvertVmCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ImportVmFromExternalProviderCommand.java A backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ImportVmFromOvaCommand.java A backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/action/ConvertOvaParameters.java A backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/action/ImportVmFromOvaParameters.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/action/VdcActionType.java 9 files changed, 136 insertions(+), 12 deletions(-) Approvals: Jenkins CI: Passed CI tests Omer Frenkel: Looks good to me, approved Arik Hadas: Verified -- To view, visit https://gerrit.ovirt.org/44130 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I0f76282ffb9c9a6882763dcb87259e479bfc4bf5 Gerrit-PatchSet: 6 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: Jenkins CI Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: core: query vm info from virtual appliance
by ahadas＠redhat.com 04 Aug '15

04 Aug '15

Arik Hadas has submitted this change and it was merged. Change subject: core: query vm info from virtual appliance ...................................................................... core: query vm info from virtual appliance Introduce a query for getting VM configuration from OVA file. Change-Id: I1ccf7b9bf33fae79b09e205fbaa4747a90be1358 Signed-off-by: Arik Hadas <ahadas(a)redhat.com> --- A backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/GetVmFromOvaQuery.java A backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/queries/GetVmFromOvaQueryParameters.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/queries/VdcQueryType.java 3 files changed, 69 insertions(+), 0 deletions(-) Approvals: Jenkins CI: Passed CI tests Omer Frenkel: Looks good to me, approved Arik Hadas: Verified -- To view, visit https://gerrit.ovirt.org/43619 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1ccf7b9bf33fae79b09e205fbaa4747a90be1358 Gerrit-PatchSet: 8 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: Jenkins CI Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: dal: Increase host hooks size limit in DB
by ofrenkel＠redhat.com 04 Aug '15

04 Aug '15

Omer Frenkel has submitted this change and it was merged. Change subject: dal: Increase host hooks size limit in DB ...................................................................... dal: Increase host hooks size limit in DB When installing all hooks on the host the returned string from VDSM has > 4000 characters which caused SQLException in HostMonitoring. Changed the type of the hooks column from VARCHAR(4000) to TEXT. Change-Id: I08851ff038145454ded278377e4c43a0e9d27948 Signed-off-by: Martin Betak <mbetak(a)redhat.com> --- A packaging/dbscripts/upgrade/03_06_1770_change_vds_dynamic_hooks_to_text.sql M packaging/dbscripts/vds_sp.sql 2 files changed, 3 insertions(+), 2 deletions(-) Approvals: Eli Mesika: Looks good to me, but someone else must approve Jenkins CI: Passed CI tests Martin Betak: Verified Omer Frenkel: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/44031 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I08851ff038145454ded278377e4c43a0e9d27948 Gerrit-PatchSet: 7 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Martin Betak <mbetak(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Betak <mbetak(a)redhat.com> Gerrit-Reviewer: Martin Polednik <mpolednik(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Shahar Havivi <shavivi(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: engine: update the attachments in case the host is moved to ...
by masayag＠redhat.com 04 Aug '15

04 Aug '15

Moti Asayag has submitted this change and it was merged. Change subject: engine: update the attachments in case the host is moved to another cluster ...................................................................... engine: update the attachments in case the host is moved to another cluster Change-Id: Ifee4f8f105c40b458e18f013fc2b508cd47c8568 Signed-off-by: Alona Kaplan <alkaplan(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ChangeVDSClusterCommand.java 1 file changed, 18 insertions(+), 0 deletions(-) Approvals: Alona Kaplan: Verified Jenkins CI: Passed CI tests Moti Asayag: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/44330 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ifee4f8f105c40b458e18f013fc2b508cd47c8568 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Alona Kaplan <alkaplan(a)redhat.com> Gerrit-Reviewer: Alona Kaplan <alkaplan(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: engine: fixing VmNetworkInterface.equals(Object) method
by masayag＠redhat.com 04 Aug '15

04 Aug '15

Moti Asayag has submitted this change and it was merged. Change subject: engine: fixing VmNetworkInterface.equals(Object) method ...................................................................... engine: fixing VmNetworkInterface.equals(Object) method Change-Id: Ib5fa770ab78ac8a1e379c8736b0140a30b65a1a7 Signed-off-by: Eliraz Levi <elevi(a)redhat.com> --- M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/network/VmNetworkInterface.java 1 file changed, 3 insertions(+), 3 deletions(-) Approvals: Jenkins CI: Passed CI tests Moti Asayag: Verified; Looks good to me, approved Lior Vernia: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/33546 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib5fa770ab78ac8a1e379c8736b0140a30b65a1a7 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Eliraz Levi <elevi(a)redhat.com> Gerrit-Reviewer: Alona Kaplan <alkaplan(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Lior Vernia <lvernia(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: core: fix in support for deprecating config values.
by masayag＠redhat.com 04 Aug '15

04 Aug '15

Moti Asayag has submitted this change and it was merged. Change subject: core: fix in support for deprecating config values. ...................................................................... core: fix in support for deprecating config values. value of deprecated ConfigKey cant be queried. Change-Id: I83c66e24552ab498e1a21b8d957ee3ee4534972d Signed-off-by: Martin Mucha <mmucha(a)redhat.com> --- M backend/manager/tools/src/main/java/org/ovirt/engine/core/config/EngineConfigLogic.java 1 file changed, 9 insertions(+), 0 deletions(-) Approvals: Martin Mucha: Verified Jenkins CI: Passed CI tests Moti Asayag: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/34692 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I83c66e24552ab498e1a21b8d957ee3ee4534972d Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Martin Mucha <mmucha(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Mucha <mmucha(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0