March 2016 - Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: userportal: added data-status attr for automation
by gshereme＠redhat.com 18 Mar '16

18 Mar '16

Greg Sheremeta has submitted this change and it was merged. Change subject: userportal: added data-status attr for automation ...................................................................... userportal: added data-status attr for automation Added a 'data-status' attribute to VM Status columns in userportal. This will allow automation / selenium / etc. to easily access the state of the VM. Change-Id: Ic4ab4ac7d9243414645dfb6f3ed895916b5a204b Bug-Url: https://bugzilla.redhat.com/1273970 Signed-off-by: Greg Sheremeta <gshereme(a)redhat.com> --- M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/ImageWithDecorator.java M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/table/cell/DecoratedImageResourceCell.java M frontend/webadmin/modules/userportal-gwtp/src/main/java/org/ovirt/engine/ui/userportal/widget/table/column/VmStatusColumn.java 3 files changed, 19 insertions(+), 5 deletions(-) Approvals: Alexander Wels: Looks good to me, approved Jenkins CI: Passed CI tests Greg Sheremeta: Verified -- To view, visit https://gerrit.ovirt.org/54842 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic4ab4ac7d9243414645dfb6f3ed895916b5a204b Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Greg Sheremeta <gshereme(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]: webadmin: Check box tooltip
by awels＠redhat.com 18 Mar '16

18 Mar '16

Alexander Wels has submitted this change and it was merged. Change subject: webadmin: Check box tooltip ...................................................................... webadmin: Check box tooltip - Fixed issue where disabling a checkbox would not enable the tooltip associated with it. - Fixed spelling error. Change-Id: Icab9605c9308086faffc5f7ebcd31ac9e6a67794 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1314181 Signed-off-by: Alexander Wels <awels(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/AbstractDiskModel.java 1 file changed, 7 insertions(+), 5 deletions(-) Approvals: Alexander Wels: Verified Jenkins CI: Passed CI tests Greg Sheremeta: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/54928 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Icab9605c9308086faffc5f7ebcd31ac9e6a67794 Gerrit-PatchSet: 2 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: Allon Mureinik <amureini(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: Vojtech Szocs <vszocs(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: userportal, webadmin: fix vm status tooltip w/ alerts
by gshereme＠redhat.com 18 Mar '16

18 Mar '16

Greg Sheremeta has submitted this change and it was merged. Change subject: userportal, webadmin: fix vm status tooltip w/ alerts ...................................................................... userportal, webadmin: fix vm status tooltip w/ alerts When an alert status icon was showing alongside the vm status icon, no indication of the alert was shown in the tooltip. Fixed. Change-Id: Ibc9d2e7a50eb938859e8eeeaaac536daacc897f7 Bug-Url: https://bugzilla.redhat.com/1304729 Signed-off-by: Greg Sheremeta <gshereme(a)redhat.com> --- M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/widget/table/cell/VmStatusCell.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/widget/table/column/VmStatusColumn.java 2 files changed, 73 insertions(+), 64 deletions(-) Approvals: Alexander Wels: Looks good to me, approved Jenkins CI: Passed CI tests Greg Sheremeta: Verified -- To view, visit https://gerrit.ovirt.org/54841 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ibc9d2e7a50eb938859e8eeeaaac536daacc897f7 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Greg Sheremeta <gshereme(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: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: Vojtech Szocs <vszocs(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-node-plugin-vdsm[master]: spec: Only use vhostmd in RHEL
by dougsland＠redhat.com 18 Mar '16

18 Mar '16

Douglas Schilling Landgraf has submitted this change and it was merged. Change subject: spec: Only use vhostmd in RHEL ...................................................................... spec: Only use vhostmd in RHEL Change-Id: I42888fa3661a64edeaf20e162557fe4c6e3e8229 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1314390 Signed-off-by: Fabian Deutsch <fabiand(a)fedoraproject.org> Reviewed-on: https://gerrit.ovirt.org/54929 Reviewed-by: Douglas Schilling Landgraf <dougsland(a)redhat.com> Continuous-Integration: Douglas Schilling Landgraf <dougsland(a)redhat.com> Tested-by: Douglas Schilling Landgraf <dougsland(a)redhat.com> --- M ovirt-node-plugin-vdsm.spec.in 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Douglas Schilling Landgraf: Verified; Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/54929 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I42888fa3661a64edeaf20e162557fe4c6e3e8229 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-node-plugin-vdsm Gerrit-Branch: master Gerrit-Owner: Fabian Deutsch <fabiand(a)redhat.com> Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[ovirt-engine-3.6]: utils: Add missing javax.activation.api dependency
by piotr.kliczewski＠gmail.com 18 Mar '16

18 Mar '16

Piotr Kliczewski has submitted this change and it was merged. Change subject: utils: Add missing javax.activation.api dependency ...................................................................... utils: Add missing javax.activation.api dependency Adds missing javax.activation.api dependency into module.xml which caused error during engine deploy on WildFly 10.0.0.GA. Change-Id: I91dcc6a2062b10fbdc1737d4cf5267cbdd936fb1 Signed-off-by: Martin Perina <mperina(a)redhat.com> --- M backend/manager/modules/utils/src/main/modules/org/ovirt/engine/core/utils/main/module.xml 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, but someone else must approve Martin Peřina: Verified; Looks good to me, approved Juan Hernandez: Looks good to me, but someone else must approve Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/54914 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I91dcc6a2062b10fbdc1737d4cf5267cbdd936fb1 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6 Gerrit-Owner: Martin Peřina <mperina(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: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com>

1 0

Change in ovirt-release[ovirt-3.6]: build: post ovirt-release36-005
by sbonazzo＠redhat.com 18 Mar '16

18 Mar '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: build: post ovirt-release36-005 ...................................................................... build: post ovirt-release36-005 Change-Id: I79499a3ea0590d1580f2818b209dfcba1a9db6f5 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M automation/build-artifacts.sh M automation/check-patch.sh M configure.ac 3 files changed, 9 insertions(+), 3 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/54925 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I79499a3ea0590d1580f2818b209dfcba1a9db6f5 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-release Gerrit-Branch: ovirt-3.6 Gerrit-Owner: Sandro Bonazzola <sbonazzo(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-3.6]: build: ovirt-release36-005
by sbonazzo＠redhat.com 18 Mar '16

18 Mar '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: build: ovirt-release36-005 ...................................................................... build: ovirt-release36-005 Change-Id: Ibddf3021dabc7e17a41d666f7d0b35343e48a122 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M automation/build-artifacts.sh M automation/check-patch.sh M configure.ac 3 files changed, 3 insertions(+), 9 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/54924 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ibddf3021dabc7e17a41d666f7d0b35343e48a122 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-release Gerrit-Branch: ovirt-3.6 Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Fabian Deutsch <fabiand(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-sdk-ruby[master]: Fix default value of "insecure" connection parameter
by juan.hernandez＠redhat.com 18 Mar '16

18 Mar '16

Juan Hernandez has submitted this change and it was merged. Change subject: Fix default value of "insecure" connection parameter ...................................................................... Fix default value of "insecure" connection parameter By default the connections are secured, and the certificates of the server are checked, but the documentation of the "insecure" parameter suggest the contrary. This patch fixes that inconsistency. Change-Id: Ie1f1528ac095b7a3b336af8c01dbb4edfb8cf62a Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M sdk/lib/ovirtsdk4/http.rb 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/54921 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie1f1528ac095b7a3b336af8c01dbb4edfb8cf62a Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-sdk-ruby 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: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: utils: Add missing javax.activation.api dependency
by mperina＠redhat.com 18 Mar '16

18 Mar '16

Martin Peřina has submitted this change and it was merged. Change subject: utils: Add missing javax.activation.api dependency ...................................................................... utils: Add missing javax.activation.api dependency Adds missing javax.activation.api dependency into module.xml which caused error during engine deploy on WildFly 10.0.0.GA. Change-Id: I91dcc6a2062b10fbdc1737d4cf5267cbdd936fb1 Signed-off-by: Martin Perina <mperina(a)redhat.com> --- M backend/manager/modules/utils/src/main/modules/org/ovirt/engine/core/utils/main/module.xml 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, but someone else must approve Martin Peřina: Verified; Looks good to me, approved Juan Hernandez: Looks good to me, but someone else must approve Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/54913 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I91dcc6a2062b10fbdc1737d4cf5267cbdd936fb1 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Martin Peřina <mperina(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: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine-sdk[master]: Use only query parameters
by juan.hernandez＠redhat.com 18 Mar '16

18 Mar '16

Juan Hernandez has submitted this change and it was merged. Change subject: Use only query parameters ...................................................................... Use only query parameters Currently the SDK has to send parameters as both query and matrix parameters, as the server some times requires query and some times matrix. But the server has recently been fixed to support both, for all paramers, so the SDK no longer needs to send both. This patch changes the SDK so that it only uses query parameters. Change-Id: I41f0aa2a0bb4b8b20e71a64769a352ea508c7583 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M generator/src/main/java/org/ovirt/sdk/python/ServicesGenerator.java M sdk/lib/ovirtsdk4/http.py 2 files changed, 7 insertions(+), 27 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/54780 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I41f0aa2a0bb4b8b20e71a64769a352ea508c7583 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine-sdk 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: gerrit-hooks <automation(a)ovirt.org>

1 0