August 2015 - Engine-commits - oVirt List Archives

Change in ovirt-reports[master]: packaging: reports-tool: text
by ydary＠redhat.com 05 Aug '15

05 Aug '15

Yaniv Dary has submitted this change and it was merged. Change subject: packaging: reports-tool: text ...................................................................... packaging: reports-tool: text Jasperreports -> JasperReports-Server Change-Id: I62b85fb6624687a946ad82ed29d3f65e58dbbe9c Bug-Url: https://bugzilla.redhat.com/1232825 Signed-off-by: Yedidyah Bar David <didi(a)redhat.com> --- M packaging/setup/plugins/ovirt-engine-reports-tool/ovirt-engine-reports/jasper/change_admin_password.py 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, but someone else must approve Jenkins CI: Passed CI tests Yaniv Dary: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/44276 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I62b85fb6624687a946ad82ed29d3f65e58dbbe9c Gerrit-PatchSet: 1 Gerrit-Project: ovirt-reports Gerrit-Branch: master Gerrit-Owner: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in jenkins[master]: added hosted-engine-setup 3.6 jobs
by sbonazzo＠redhat.com 05 Aug '15

05 Aug '15

Sandro Bonazzola has submitted this change and it was merged. Change subject: added hosted-engine-setup 3.6 jobs ...................................................................... added hosted-engine-setup 3.6 jobs Change-Id: I8ce50ac8847fb477088dda40277f8536cb11f3eb Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M jobs/confs/yaml/jobs/ovirt-hosted-engine-setup/ovirt-hosted-engine-setup_create-rpms.yaml M jobs/confs/yaml/jobs/ovirt/publish-rpms-nightly-3.6.yaml 2 files changed, 6 insertions(+), 1 deletion(-) Approvals: Sandro Bonazzola: Verified; Looks good to me; Ready for merge Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/44410 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8ce50ac8847fb477088dda40277f8536cb11f3eb Gerrit-PatchSet: 1 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>

1 0

Change in ovirt-engine[master]: restapi: set storageDomainId only when available on VmDisksR...
by derez＠redhat.com 05 Aug '15

05 Aug '15

Daniel Erez has submitted this change and it was merged. Change subject: restapi: set storageDomainId only when available on VmDisksResource ...................................................................... restapi: set storageDomainId only when available on VmDisksResource BackendVmDisksResource -> getAddParameters: set storageDomainId only when available to allow auto-select of a storage domain on 'AddDiskCommand -> getStorageDomainId() -> getDisksStorageDomainId()'. The behaviour is needed for backwards compatibility of MSD (multiple storage domains) feature. Change-Id: I2857350bafad327a1e76b848165969368f4213df Bug-Url: https://bugzilla.redhat.com/1220824 Signed-off-by: Daniel Erez <derez(a)redhat.com> --- M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendVmDisksResource.java 1 file changed, 4 insertions(+), 1 deletion(-) Approvals: Ori Liel: Looks good to me, approved Jenkins CI: Passed CI tests Daniel Erez: Verified -- To view, visit https://gerrit.ovirt.org/44270 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2857350bafad327a1e76b848165969368f4213df Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Ori Liel <oliel(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: frontend: Localize SerialNumberPolicy enum
by tjelinek＠redhat.com 05 Aug '15

05 Aug '15

Tomas Jelinek has submitted this change and it was merged. Change subject: frontend: Localize SerialNumberPolicy enum ...................................................................... frontend: Localize SerialNumberPolicy enum Moved SerialNumberPolicy values from Enums -> LocalizedEnums so they could be properly translated. Change-Id: I6b0a1c6d23c6df77ddece6de0590a143fa3d3909 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1247889 Signed-off-by: Martin Betak <mbetak(a)redhat.com> --- M frontend/webadmin/modules/uicompat/src/main/java/org/ovirt/engine/ui/uicompat/Enums.java M frontend/webadmin/modules/uicompat/src/main/java/org/ovirt/engine/ui/uicompat/LocalizedEnums.java M frontend/webadmin/modules/uicompat/src/main/resources/org/ovirt/engine/ui/uicompat/Enums.properties M frontend/webadmin/modules/uicompat/src/main/resources/org/ovirt/engine/ui/uicompat/LocalizedEnums.properties 4 files changed, 9 insertions(+), 9 deletions(-) Approvals: Tomas Jelinek: Looks good to me, approved Jenkins CI: Passed CI tests Martin Betak: Verified -- To view, visit https://gerrit.ovirt.org/44399 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6b0a1c6d23c6df77ddece6de0590a143fa3d3909 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Martin Betak <mbetak(a)redhat.com> Gerrit-Reviewer: Einav Cohen <ecohen(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Betak <mbetak(a)redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in jenkins[master]: 3.6 branching - updated ovirt-engine builder job
by sbonazzo＠redhat.com 05 Aug '15

05 Aug '15

Sandro Bonazzola has submitted this change and it was merged. Change subject: 3.6 branching - updated ovirt-engine builder job ...................................................................... 3.6 branching - updated ovirt-engine builder job Updated the ovirt-engine builder job to build the ovirt-engine-3.6 branch, and updated the 3.6 publisher to publish its output. Change-Id: I7663d8e8c03e24ed932856b4cafe495df918e49d Signed-off-by: Sharon Naftaly <snaftaly(a)redhat.com> --- M jobs/confs/yaml/jobs/ovirt-engine/ovirt-engine_build-artifacts.yaml M jobs/confs/yaml/jobs/ovirt/publish-rpms-nightly-3.6.yaml 2 files changed, 3 insertions(+), 1 deletion(-) Approvals: Sandro Bonazzola: Verified; Looks good to me; Ready for merge Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/44398 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I7663d8e8c03e24ed932856b4cafe495df918e49d Gerrit-PatchSet: 3 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Sharon Naftaly <snaftaly(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>

1 0

Change in ovirt-engine[master]: webadmin, userportal: Cache of VmConsoles removed
by tjelinek＠redhat.com 05 Aug '15

05 Aug '15

Tomas Jelinek has submitted this change and it was merged. Change subject: webadmin, userportal: Cache of VmConsoles removed ...................................................................... webadmin, userportal: Cache of VmConsoles removed VmConsoles objects are not cached, rather created on demand thus they can be well configured all the time. The bug was caused by resolving wrong console type during first list show (when the vms were down) and keeping that settings even after vms were capable of non-rdp connection. Signed-off-by: Jakub Niedermertl <jniederm(a)redhat.com> Change-Id: I92fd4807a9d2daea67b8ba547c75717a1764268d Bug-Url: https://bugzilla.redhat.com/1236674 --- D frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/ConsoleModelsCache.java A frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/ConsolesFactory.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/userportal/AbstractUserPortalListModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/userportal/UserPortalBasicListModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/userportal/UserPortalItemModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/userportal/UserPortalListModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/VmListModel.java 7 files changed, 102 insertions(+), 165 deletions(-) Approvals: Tomas Jelinek: Looks good to me, approved Jakub Niedermertl: Verified Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/44179 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I92fd4807a9d2daea67b8ba547c75717a1764268d Gerrit-PatchSet: 7 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Jakub Niedermertl <jniederm(a)redhat.com> Gerrit-Reviewer: Jakub Niedermertl <jniederm(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Betak <mbetak(a)redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in jenkins[master]: 3.6 branching
by sbonazzo＠redhat.com 05 Aug '15

05 Aug '15

Sandro Bonazzola has submitted this change and it was merged. Change subject: 3.6 branching ...................................................................... 3.6 branching - publish nightly 3.6 - added 3.6 jobs for already branched projects Change-Id: Ib912604460757a80fee92c4ee0c85b5dc0c8e72d Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M jobs/confs/yaml/jobs/ovirt-image-uploader/ovirt-image-uploader_standard.yaml M jobs/confs/yaml/jobs/ovirt-iso-uploader/ovirt-iso-uploader_create-rpms.yaml M jobs/confs/yaml/jobs/ovirt-live/ovirt-live_build-artifacts.yaml M jobs/confs/yaml/jobs/ovirt-log-collector/ovirt-log-collector_create-rpms.yaml A jobs/confs/yaml/jobs/ovirt/publish-rpms-nightly-3.6.yaml 5 files changed, 175 insertions(+), 0 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me; Ready for merge Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/44355 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib912604460757a80fee92c4ee0c85b5dc0c8e72d Gerrit-PatchSet: 4 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Anton Marchukov <amarchuk(a)redhat.com> Gerrit-Reviewer: Barak Korren <bkorren(a)redhat.com> Gerrit-Reviewer: David Caro <dcaroest(a)redhat.com> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Lev Veyde <lveyde(a)redhat.com> Gerrit-Reviewer: Max Kovgan <mvk(a)redhat.com> Gerrit-Reviewer: Sagi Shnaidman <sshnaidm(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Sharon Naftaly <snaftaly(a)redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com>

1 0

Change in ovirt-engine[master]: core: toString method for VmPool
by ofrenkel＠redhat.com 05 Aug '15

05 Aug '15

Omer Frenkel has submitted this change and it was merged. Change subject: core: toString method for VmPool ...................................................................... core: toString method for VmPool toString() method of VmPool introduced to simplify debugging. The implementation follows VM class. Change-Id: I2db8556c1bc292b279ecbde65e6afa609dec7120 Signed-off-by: Jakub Niedermertl <jniederm(a)redhat.com> --- M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VmPool.java 1 file changed, 5 insertions(+), 0 deletions(-) Approvals: Jakub Niedermertl: Verified Jenkins CI: Passed CI tests Omer Frenkel: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/44360 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2db8556c1bc292b279ecbde65e6afa609dec7120 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Jakub Niedermertl <jniederm(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Jakub Niedermertl <jniederm(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-log-collector[master]: build: ovirt-log-collector-4.0.0_master
by sbonazzo＠redhat.com 05 Aug '15

05 Aug '15

Sandro Bonazzola has submitted this change and it was merged. Change subject: build: ovirt-log-collector-4.0.0_master ...................................................................... build: ovirt-log-collector-4.0.0_master Change-Id: I5fa678eed19bc90aeb5b3ca1d9b117303659e375 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M configure.ac M ovirt-log-collector.spec.in 2 files changed, 5 insertions(+), 8 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/44396 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5fa678eed19bc90aeb5b3ca1d9b117303659e375 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-log-collector Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: backend: Fix incorrect allocation of Host Devices
by ofrenkel＠redhat.com 05 Aug '15

05 Aug '15

Omer Frenkel has submitted this change and it was merged. Change subject: backend: Fix incorrect allocation of Host Devices ...................................................................... backend: Fix incorrect allocation of Host Devices It was possible for the MarkHostDevicesUsedByVmId to accidentally allocate same host device on multiple hosts if there existed such host devices in DB. This patch adds the host id to the stored procedure parameter to ensure the allocation is only performed on given host's devices. Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1250363 Change-Id: I390452251e33b487a5d815c71d8a29958ff23db8 Signed-off-by: Martin Betak <mbetak(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/hostdev/HostDeviceManager.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/HostDeviceDao.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/HostDeviceDaoImpl.java M backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/HostDeviceDaoTest.java M packaging/dbscripts/host_device_sp.sql 6 files changed, 12 insertions(+), 9 deletions(-) Approvals: Jenkins CI: Passed CI tests Martin Betak: Verified Omer Frenkel: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/44364 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I390452251e33b487a5d815c71d8a29958ff23db8 Gerrit-PatchSet: 6 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: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0