July 2016 - Engine-commits - oVirt List Archives

Change in ovirt-engine[ovirt-engine-4.0.2]: userportal: User is not logged out from userportal on sessio...
by tnisan＠redhat.com 21 Jul '16

21 Jul '16

Tal Nisan has submitted this change and it was merged. Change subject: userportal: User is not logged out from userportal on session timeout ...................................................................... userportal: User is not logged out from userportal on session timeout User should be logged out of UserPortal on session timeout. Change-Id: Ia8a3e342866083646e626c03ca0997920e9b7f3c Bug-Url: https://bugzilla.redhat.com/1357452 Signed-off-by: Ravi Nori <rnori(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/QueriesCommandBase.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/dataprovider/AsyncDataProvider.java 2 files changed, 4 insertions(+), 2 deletions(-) Approvals: Martin Peřina: Looks good to me, approved Ravi Nori: Verified Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/61147 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia8a3e342866083646e626c03ca0997920e9b7f3c Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.0.2 Gerrit-Owner: Ravi Nori <rnori(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Ravi Nori <rnori(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: core: update interface statistics not thorugh resource manager
by ahadas＠redhat.com 21 Jul '16

21 Jul '16

Arik Hadas has submitted this change and it was merged. Change subject: core: update interface statistics not thorugh resource manager ...................................................................... core: update interface statistics not thorugh resource manager VmAnalyzer remained the only place that rely on the internalSetVmStatus in ResourceManager to update VM network interface statistics. This patch changes it so we could eventually remove this part from the internalSetVmStatus method. Change-Id: Ia9c02d7ec774ab444f84d87a0d6c8664eb49d62a Signed-off-by: Arik Hadas <ahadas(a)redhat.com> --- M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/network/NetworkStatistics.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/ResourceManager.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/monitoring/VmAnalyzer.java M backend/manager/modules/vdsbroker/src/test/java/org/ovirt/engine/core/vdsbroker/monitoring/VmAnalyzerTest.java 4 files changed, 15 insertions(+), 12 deletions(-) Approvals: Shahar Havivi: Looks good to me, approved Jenkins CI: Passed CI tests Arik Hadas: Verified -- To view, visit https://gerrit.ovirt.org/60927 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia9c02d7ec774ab444f84d87a0d6c8664eb49d62a Gerrit-PatchSet: 7 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: Shahar Havivi <shavivi(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: core: change the way network interfaces are stored in vm-ana...
by ahadas＠redhat.com 21 Jul '16

21 Jul '16

Arik Hadas has submitted this change and it was merged. Change subject: core: change the way network interfaces are stored in vm-analyzer ...................................................................... core: change the way network interfaces are stored in vm-analyzer Save VM network interfaces as a field in VmAnalyzer instead of inside the VM object that represents the VM that was queried from the DB. This will allow us to query only the dynamic data later on. Change-Id: I72789ed13bce93bf3b4a0daddebbc629fa8c2803 Signed-off-by: Arik Hadas <ahadas(a)redhat.com> --- M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/monitoring/VmAnalyzer.java 1 file changed, 6 insertions(+), 5 deletions(-) Approvals: Shahar Havivi: Looks good to me, approved Arik Hadas: Verified; Passed CI tests -- To view, visit https://gerrit.ovirt.org/60926 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I72789ed13bce93bf3b4a0daddebbc629fa8c2803 Gerrit-PatchSet: 7 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: Shahar Havivi <shavivi(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: core: load network interfaces in vm-analyzer
by ahadas＠redhat.com 21 Jul '16

21 Jul '16

Arik Hadas has submitted this change and it was merged. Change subject: core: load network interfaces in vm-analyzer ...................................................................... core: load network interfaces in vm-analyzer Move the load of VM network interfaces from VmAnalyzerFactory to VmAnalyzer. Change-Id: I79d09424a20683ecd65a1898befc79f5ea64967c Signed-off-by: Arik Hadas <ahadas(a)redhat.com> --- M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/monitoring/VmAnalyzer.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/monitoring/VmAnalyzerFactory.java M backend/manager/modules/vdsbroker/src/test/java/org/ovirt/engine/core/vdsbroker/monitoring/VmAnalyzerTest.java 3 files changed, 18 insertions(+), 24 deletions(-) Approvals: Shahar Havivi: Looks good to me, approved Jenkins CI: Passed CI tests Arik Hadas: Verified -- To view, visit https://gerrit.ovirt.org/60925 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I79d09424a20683ecd65a1898befc79f5ea64967c Gerrit-PatchSet: 7 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: Shahar Havivi <shavivi(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: webadmin: remove tooltip underline from table column headers
by gshereme＠redhat.com 21 Jul '16

21 Jul '16

Greg Sheremeta has submitted this change and it was merged. Change subject: webadmin: remove tooltip underline from table column headers ...................................................................... webadmin: remove tooltip underline from table column headers Removed underlines from table column headers that have tool tips. This provides consistency with PatternFly in the UX. Change-Id: Ic1f393ec5f63580f928b8b9eacbe32ad2b04cf10 Bug-Url: https://bugzilla.redhat.com/1230159 Signed-off-by: Scott J Dickerson <sdickers(a)redhat.com> --- M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/table/header/AbstractCheckboxHeader.java M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/table/header/AbstractHeader.java M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/table/header/SafeHtmlHeader.java 3 files changed, 7 insertions(+), 30 deletions(-) Approvals: Jenkins CI: Passed CI tests Greg Sheremeta: Verified; Looks good to me, approved Scott Dickerson: Verified -- To view, visit https://gerrit.ovirt.org/61000 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic1f393ec5f63580f928b8b9eacbe32ad2b04cf10 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Scott Dickerson <sdickers(a)redhat.com> Gerrit-Reviewer: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Greg Sheremeta <gshereme(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Scott Dickerson <sdickers(a)redhat.com> Gerrit-Reviewer: Vojtech Szocs <vszocs(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: restapi: Fix documentation of template disk remove
by juan.hernandez＠redhat.com 21 Jul '16

21 Jul '16

Juan Hernandez has submitted this change and it was merged. Change subject: restapi: Fix documentation of template disk remove ...................................................................... restapi: Fix documentation of template disk remove The operation that removes a disk from a template has moved from `disks` service to the new `diskattachments` service, but this change isn't reflected in the documentation. This patch improves the documentation so that it explicitly describes this change. Change-Id: Ifb0f2c17270a438ff8617e662c4e21c7e0d8269c Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M backend/manager/modules/restapi/README.adoc 1 file changed, 5 insertions(+), 4 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved Jenkins CI: Passed CI tests Vaclav Kondula: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/61140 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ifb0f2c17270a438ff8617e662c4e21c7e0d8269c Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Vaclav Kondula <vkondula(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[ovirt-engine-4.0]: userportal: User is not logged out from userportal on sessio...
by tnisan＠redhat.com 21 Jul '16

21 Jul '16

Tal Nisan has submitted this change and it was merged. Change subject: userportal: User is not logged out from userportal on session timeout ...................................................................... userportal: User is not logged out from userportal on session timeout User should be logged out of UserPortal on session timeout. Change-Id: Ia8a3e342866083646e626c03ca0997920e9b7f3c Bug-Url: https://bugzilla.redhat.com/1357452 Signed-off-by: Ravi Nori <rnori(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/QueriesCommandBase.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/dataprovider/AsyncDataProvider.java 2 files changed, 4 insertions(+), 2 deletions(-) Approvals: Martin Peřina: Looks good to me, approved Ravi Nori: Verified Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/61098 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia8a3e342866083646e626c03ca0997920e9b7f3c Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.0 Gerrit-Owner: Ravi Nori <rnori(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Ravi Nori <rnori(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[ovirt-engine-4.0]: engine: host upgrade should upgrade all oVirt related packages
by tnisan＠redhat.com 21 Jul '16

21 Jul '16

Tal Nisan has submitted this change and it was merged. Change subject: engine: host upgrade should upgrade all oVirt related packages ...................................................................... engine: host upgrade should upgrade all oVirt related packages Host upgrade to detect and upgrade all relevant packages not just vdsm and vdsm-cli. The current list of packages checked are ioprocess,mom,libvirt,lvm2,ovirt-imageio-common, ovirt-imageio-daemon,ovirt-vmconsole, ovirt-vmconsole-host,python-ioprocess, qemu-kvm,qemu-img,sanlock,vdsm,vdsm-cli Change-Id: Ia50658dcb875c18e4382086dbc5e22ea7918d2e5 Bug-Url: https://bugzilla.redhat.com/1344020 Signed-off-by: Ravi Nori <rnori(a)redhat.com> --- M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/config/ConfigValues.java M packaging/dbscripts/upgrade/pre_upgrade/0000_config.sql 2 files changed, 2 insertions(+), 1 deletion(-) Approvals: Martin Peřina: Looks good to me, approved Ravi Nori: Verified Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/61099 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia50658dcb875c18e4382086dbc5e22ea7918d2e5 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.0 Gerrit-Owner: Ravi Nori <rnori(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Ravi Nori <rnori(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-live[ovirt-live-4.0]: build: oVirt Live 4.0.2_rc1
by sbonazzo＠redhat.com 20 Jul '16

20 Jul '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: build: oVirt Live 4.0.2_rc1 ...................................................................... build: oVirt Live 4.0.2_rc1 Change-Id: I4c32cbbf581f283aaece327946f08e6ea685a287 Signed-off-by: Lev Veyde <lveyde(a)redhat.com> --- M centos-7/Makefile M centos-7/kickstart/ovirt-live-ovirt-custom.ks.in 2 files changed, 2 insertions(+), 2 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/61012 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4c32cbbf581f283aaece327946f08e6ea685a287 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-live Gerrit-Branch: ovirt-live-4.0 Gerrit-Owner: Lev Veyde <lveyde(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-release[ovirt-4.0]: build: ovirt-release40-4.0.2
by sbonazzo＠redhat.com 20 Jul '16

20 Jul '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: build: ovirt-release40-4.0.2 ...................................................................... build: ovirt-release40-4.0.2 Change-Id: Idf8cd1ab5218519c0cbee08890c9873cdf450311 --- M automation/build-artifacts.sh M configure.ac M ovirt-release40.spec.in 3 files changed, 25 insertions(+), 23 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/61097 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Idf8cd1ab5218519c0cbee08890c9873cdf450311 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-release Gerrit-Branch: ovirt-4.0 Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Fabian Deutsch <fabiand(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Ryan Barry <rbarry(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0