- Engine-commits - oVirt List Archives

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

25 Jul '16

Oved Ourfali 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: Looks good to me, approved Scott Dickerson: Verified -- To view, visit https://gerrit.ovirt.org/61215 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic1f393ec5f63580f928b8b9eacbe32ad2b04cf10 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.0 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: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Vojtech Szocs <vszocs(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in jenkins[master]: yamlize the scan_security_and_license_ovirt-engine_master job
by eedri＠redhat.com 25 Jul '16

25 Jul '16

Eyal Edri has submitted this change and it was merged. Change subject: yamlize the scan_security_and_license_ovirt-engine_master job ...................................................................... yamlize the scan_security_and_license_ovirt-engine_master job Change-Id: Ia2f05310897787234079612db8839a36dddf7010 Signed-off-by: Sharon Naftaly <snaftaly(a)redhat.com> --- A jobs/confs/yaml/jobs/scan-security-and-license-ovirt-engine.yaml 1 file changed, 39 insertions(+), 0 deletions(-) Approvals: Sharon Naftaly: Verified; Ready for review Eyal Edri: Looks good to me; Ready for merge Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/60165 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia2f05310897787234079612db8839a36dddf7010 Gerrit-PatchSet: 4 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Sharon Naftaly <snaftaly(a)redhat.com> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sharon Naftaly <snaftaly(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in jenkins[master]: remove emails from setup-upgrade job
by eedri＠redhat.com 25 Jul '16

25 Jul '16

Eyal Edri has submitted this change and it was merged. Change subject: remove emails from setup-upgrade job ...................................................................... remove emails from setup-upgrade job most jobs doesn't email so not reason to keep this specific job, since we monitor the unstable queue constantly and also we have daily unstable report sent to infra. In addition, this job was added to per patch so we don't want emails sent on every patch failing Change-Id: I57fbba9d30296af552392cf730cc3011dc946a12 Signed-off-by: Eyal Edri <eedri(a)redhat.com> --- M jobs/confs/yaml/templates/engine-upgrade.yaml 1 file changed, 0 insertions(+), 7 deletions(-) Approvals: Eyal Edri: Verified; Ready for merge Barak Korren: Looks good to me Gil Shinar: Looks good to me Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/61176 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I57fbba9d30296af552392cf730cc3011dc946a12 Gerrit-PatchSet: 1 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Anton Marchukov <amarchuk(a)redhat.com> Gerrit-Reviewer: Barak Korren <bkorren(a)redhat.com> Gerrit-Reviewer: David Caro <david(a)dcaro.es> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Gil Shinar <gshinar(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-engine[master]: core: Removed duplicate LOW_UILIZATION parameter
by rgolan＠redhat.com 24 Jul '16

24 Jul '16

Roy Golan has submitted this change and it was merged. Change subject: core: Removed duplicate LOW_UILIZATION parameter ...................................................................... core: Removed duplicate LOW_UILIZATION parameter The LOW_UTILIZATION parameter appeared twice in PowerSavingBalancePolicyUnit parameters. This patch removes one of the appearances. Change-Id: Ia3b7928c72962d5f98dfe3eb969b94f9b0101f63 Bug-Url: https://bugzilla.redhat.com/1351556 Signed-off-by: Jenny Tokar <jtokar(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/scheduling/policyunits/PowerSavingBalancePolicyUnit.java 1 file changed, 0 insertions(+), 1 deletion(-) Approvals: Martin Sivák: Looks good to me, but someone else must approve Jenkins CI: Passed CI tests Roy Golan: Looks good to me, approved Jenny Tokar: Verified -- To view, visit https://gerrit.ovirt.org/61071 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia3b7928c72962d5f98dfe3eb969b94f9b0101f63 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Jenny Tokar <jtokar(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Jenny Tokar <jtokar(a)redhat.com> Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: core: Change default value of DwhHeartBeatInterval
by mperina＠redhat.com 24 Jul '16

24 Jul '16

Martin Peřina has submitted this change and it was merged. Change subject: core: Change default value of DwhHeartBeatInterval ...................................................................... core: Change default value of DwhHeartBeatInterval Lowers default value of DwhHeartBeatInterval in vdc_options from 30 seconds to 15 seconds (if users changed default value using engine-config we will preserve value defined by users). Change-Id: Iea4656e74d1bff1bdc68f1fa44d03d55febbc39b Bug-Url: https://bugzilla.redhat.com/1344935 Signed-off-by: Martin Perina <mperina(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 M packaging/etc/engine-config/engine-config.properties 3 files changed, 5 insertions(+), 2 deletions(-) Approvals: Shirly Radco: Looks good to me, but someone else must approve Martin Peřina: Verified; Looks good to me, approved Eli Mesika: Looks good to me, but someone else must approve Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/61234 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iea4656e74d1bff1bdc68f1fa44d03d55febbc39b 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: Jenkins CI Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Shirly Radco <sradco(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-dwh[ovirt-engine-dwh-4.0]: history: added disks_vm_map view
by Shirly Radco 24 Jul '16

24 Jul '16

Shirly Radco has submitted this change and it was merged. Change subject: history: added disks_vm_map view ...................................................................... history: added disks_vm_map view In oreder to temporarily fix manageIQ from braking added disks_vm_map view instead of disks_vm_map table. Change-Id: I920a5d2fa61f1a73d6eb9681c8567849ccbeccc3 Bug-Url: https://bugzilla.redhat.com/1354150 Signed-off-by: Shirly Radco <sradco(a)redhat.com> --- M packaging/dbscripts/create_views_3_6.sql 1 file changed, 10 insertions(+), 0 deletions(-) Approvals: Shirly Radco: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/60434 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I920a5d2fa61f1a73d6eb9681c8567849ccbeccc3 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-dwh Gerrit-Branch: ovirt-engine-dwh-4.0 Gerrit-Owner: Shirly Radco <sradco(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Shirly Radco <sradco(a)redhat.com> Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: webadmin: Fix scrollable tab bar width calculation
by awels＠redhat.com 22 Jul '16

22 Jul '16

Alexander Wels has submitted this change and it was merged. Change subject: webadmin: Fix scrollable tab bar width calculation ...................................................................... webadmin: Fix scrollable tab bar width calculation - The width calculation was not being triggered when a dynamic main tab was shown/hidden. This patch fixes that issues. - Queued up recalculations when a bunch of tabs change their visibility to only recalculate once. Change-Id: I2f979747216d6d9ee666c46a26c59806de08216a Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1346817 Signed-off-by: Alexander Wels <awels(a)redhat.com> --- M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/presenter/ScrollableTabBarPresenterWidget.java 1 file changed, 34 insertions(+), 6 deletions(-) Approvals: Alexander Wels: Verified Jenkins CI: Passed CI tests Vojtech Szocs: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/61118 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2f979747216d6d9ee666c46a26c59806de08216a Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Alexander Wels <awels(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: Vojtech Szocs <vszocs(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: webadmin: Disable CPU Pass-Through when Manual Migration is ...
by awels＠redhat.com 21 Jul '16

21 Jul '16

Alexander Wels has submitted this change and it was merged. Change subject: webadmin: Disable CPU Pass-Through when Manual Migration is Enabled ...................................................................... webadmin: Disable CPU Pass-Through when Manual Migration is Enabled This patch disables the 'Pass-Through Host CPU' option when manual migration is enabled as libvirt does not support the option to migrate a VM with CPU passthrough. This patch also corrects the wording of the tooltip for this option to correctly indicate that this functionality isn't supported. Change-Id: Ice8fc5556f139a39eeb352d423dfd3431cbefbcb Bug-Url: https://bugzilla.redhat.com/1342928 Signed-off-by: Phillip Bailey <phbailey(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/VmModelBehaviorBase.java M frontend/webadmin/modules/uicompat/src/main/resources/org/ovirt/engine/ui/uicompat/UIConstants.properties 2 files changed, 6 insertions(+), 22 deletions(-) Approvals: Roman Mohr: Looks good to me, but someone else must approve Alexander Wels: Looks good to me, approved Jenkins CI: Passed CI tests Phillip Bailey: Verified -- To view, visit https://gerrit.ovirt.org/60421 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ice8fc5556f139a39eeb352d423dfd3431cbefbcb Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Phillip Bailey <phbailey(a)redhat.com> Gerrit-Reviewer: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Andrej Krejcir <akrejcir(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Jenny Tokar <jtokar(a)redhat.com> Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Phillip Bailey <phbailey(a)redhat.com> Gerrit-Reviewer: Roman Mohr <rmohr(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Yanir Quinn <yquinn(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: webadmin: image upload - remove image source panel
by derez＠redhat.com 21 Jul '16

21 Jul '16

Daniel Erez has submitted this change and it was merged. Change subject: webadmin: image upload - remove image source panel ...................................................................... webadmin: image upload - remove image source panel Image Upload dialog: removed 'Image Source' panel as uploading using URI is currently not supported. Change-Id: Ia7ac2b774feedd93cc1b6b4ac76b14b02df554dd Bug-Url: https://bugzilla.redhat.com/1358729 Signed-off-by: Daniel Erez <derez(a)redhat.com> --- M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/storage/UploadImagePopupView.ui.xml 1 file changed, 2 insertions(+), 2 deletions(-) Approvals: Jenkins CI: Passed CI tests Daniel Erez: Verified; Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/61178 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia7ac2b774feedd93cc1b6b4ac76b14b02df554dd Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Amit Aviram <aaviram(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: aaa: Restapi access should create sessions only if persisten...
by mperina＠redhat.com 21 Jul '16

21 Jul '16

Martin Peřina has submitted this change and it was merged. Change subject: aaa: Restapi access should create sessions only if persistent_auth is requested ...................................................................... aaa: Restapi access should create sessions only if persistent_auth is requested Restapi authentication filters should not create sessions for bearer auth, only basic auth requests should have sessions created. Change-Id: I6f769c358e32637fa6289636542dd8548f786069 Bug-Url: https://bugzilla.redhat.com/1358434 Signed-off-by: Ravi Nori <rnori(a)redhat.com> --- M backend/manager/modules/aaa/src/main/java/org/ovirt/engine/core/aaa/SsoUtils.java M backend/manager/modules/aaa/src/main/java/org/ovirt/engine/core/aaa/filters/SsoRestApiAuthFilter.java M backend/manager/modules/aaa/src/main/java/org/ovirt/engine/core/aaa/filters/SsoRestApiNegotiationFilter.java 3 files changed, 4 insertions(+), 13 deletions(-) Approvals: Martin Peřina: Looks good to me, approved Ravi Nori: Verified Juan Hernandez: Looks good to me, but someone else must approve Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/61052 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6f769c358e32637fa6289636542dd8548f786069 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Ravi Nori <rnori(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Ravi Nori <rnori(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0