March 2016 - Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: core: Call end action once CopyTaskHandler is finished.
by mlipchuk＠redhat.com 20 Mar '16

20 Mar '16

Maor Lipchuk has submitted this change and it was merged. Change subject: core: Call end action once CopyTaskHandler is finished. ...................................................................... core: Call end action once CopyTaskHandler is finished. Call end successfully once glance image if finished to be imported so actual size will be updated properly. Change-Id: I44343b1f49d099fc6febd3907066a35f22ef3a4f Bug-Url: https://bugzilla.redhat.com/1291080 Signed-off-by: Maor Lipchuk <mlipchuk(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/repoimage/ImportRepoImageCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/repoimage/ImportRepoImageCopyTaskHandler.java 2 files changed, 4 insertions(+), 3 deletions(-) Approvals: Jenkins CI: Passed CI tests Daniel Erez: Looks good to me, approved Maor Lipchuk: Verified -- To view, visit https://gerrit.ovirt.org/54688 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I44343b1f49d099fc6febd3907066a35f22ef3a4f Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: webadmin: Warning for unsupported memory restore (Custom Sna...
by ahadas＠redhat.com 20 Mar '16

20 Mar '16

Arik Hadas has submitted this change and it was merged. Change subject: webadmin: Warning for unsupported memory restore (Custom Snapshot Preview) ...................................................................... webadmin: Warning for unsupported memory restore (Custom Snapshot Preview) The user gets warning message when trying to restore snapshot with memory which was taken in different cluster version (i.e. before upgrade). This patch is relevant to the Custom Snapshot Preview dialog. Change-Id: Id5fc98bd2478195f56fdd00fb1b9b098eef22c8a Bug-Url: https://bugzilla.redhat.com/1292398 Signed-off-by: Marek Libra <mlibra(a)redhat.com> --- M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/CommonApplicationConstants.java M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/popup/vm/VmSnapshotCustomPreviewPopupWidget.java M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/popup/vm/VmSnapshotCustomPreviewPopupWidget.ui.xml M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/SnapshotModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/VmSnapshotListModel.java 5 files changed, 49 insertions(+), 34 deletions(-) Approvals: Jenkins CI: Passed CI tests Daniel Erez: Looks good to me, approved Marek Libra: Verified Arik Hadas: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/53974 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id5fc98bd2478195f56fdd00fb1b9b098eef22c8a Gerrit-PatchSet: 7 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Marek Libra <mlibra(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Liron Aravot <laravot(a)redhat.com> Gerrit-Reviewer: Marek Libra <mlibra(a)redhat.com> Gerrit-Reviewer: Martin Betak <mbetak(a)redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: Revert "core: Migration bandwidth columns added to stored pr...
by ahadas＠redhat.com 20 Mar '16

20 Mar '16

Arik Hadas has submitted this change and it was merged. Change subject: Revert "core: Migration bandwidth columns added to stored procedures" ...................................................................... Revert "core: Migration bandwidth columns added to stored procedures" This reverts commit 8df05f9348e61f5d4b146d017859083c66b63c74. Change-Id: I2f84d7382134ea65aebdbfe23625e0e59cc8e8e1 Signed-off-by: Arik Hadas <ahadas(a)redhat.com> --- M packaging/dbscripts/clusters_sp.sql 1 file changed, 5 insertions(+), 15 deletions(-) Approvals: Jenkins CI: Passed CI tests Arik Hadas: Verified; Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/54963 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2f84d7382134ea65aebdbfe23625e0e59cc8e8e1 Gerrit-PatchSet: 1 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: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[ovirt-engine-3.6]: webadmin: Check box tooltip
by tnisan＠redhat.com 20 Mar '16

20 Mar '16

Tal Nisan 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/54944 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Icab9605c9308086faffc5f7ebcd31ac9e6a67794 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6 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: Oved Ourfali <oourfali(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 ovirt-engine[master]: core: Migration bandwidth columns added to stored procedures
by ahadas＠redhat.com 20 Mar '16

20 Mar '16

Arik Hadas has submitted this change and it was merged. Change subject: core: Migration bandwidth columns added to stored procedures ...................................................................... core: Migration bandwidth columns added to stored procedures Columns 'migration_bandwidth_limit_type' and 'custom_migration_bandwidth_limit' added to cluster Insert and Update storage procedures. Change-Id: I506050622a248644eb69a48e830231f223dccd96 Bug-Url: https://bugzilla.redhat.com/1252426 Feature-Page: http://www.ovirt.org/Features/Migration_Enhancements Signed-off-by: Jakub Niedermertl <jniederm(a)redhat.com> --- M packaging/dbscripts/clusters_sp.sql 1 file changed, 15 insertions(+), 5 deletions(-) Approvals: Jakub Niedermertl: Verified Eli Mesika: Looks good to me, but someone else must approve Jenkins CI: Passed CI tests Arik Hadas: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/54886 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I506050622a248644eb69a48e830231f223dccd96 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: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Jakub Niedermertl <jniederm(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in jenkins[master]: Remove all the log gathering from the system job
by dcaro＠redhat.com 19 Mar '16

19 Mar '16

David Caro has submitted this change and it was merged. Change subject: Remove all the log gathering from the system job ...................................................................... Remove all the log gathering from the system job Now it's the runner script that will gather it as it belongs along the code Change-Id: I6a1047eab78ca2dd6067410606725368f1bf62ba Signed-off-by: David Caro <dcaroest(a)redhat.com> --- M jobs/confs/shell-scripts/system_tests.collect_logs.sh 1 file changed, 4 insertions(+), 21 deletions(-) Approvals: Jenkins CI: Passed CI tests David Caro: Verified; Looks good to me; Ready for merge -- To view, visit https://gerrit.ovirt.org/54956 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6a1047eab78ca2dd6067410606725368f1bf62ba Gerrit-PatchSet: 2 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: David Caro <dcaro(a)redhat.com> Gerrit-Reviewer: David Caro <dcaro(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine-sdk-ruby[master]: Remove the "kerberos" test tag
by juan.hernandez＠redhat.com 18 Mar '16

18 Mar '16

Juan Hernandez has submitted this change and it was merged. Change subject: Remove the "kerberos" test tag ...................................................................... Remove the "kerberos" test tag This patch temoves the "kerberos" test tag, as all the tests, including the ones that use Kerberos, can run now using the embedded test server. Change-Id: Ic3b32be7b1e63558ad81c1f62c56b650221102ee Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M .gitignore M README.adoc D sdk/spec/clusters_kerberos_service_spec.rb M sdk/spec/connection_create_spec.rb M sdk/spec/spec_helper.rb 5 files changed, 71 insertions(+), 114 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/54946 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic3b32be7b1e63558ad81c1f62c56b650221102ee Gerrit-PatchSet: 3 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-sdk-ruby[master]: Add embedded web server for tests
by juan.hernandez＠redhat.com 18 Mar '16

18 Mar '16

Juan Hernandez has submitted this change and it was merged. Change subject: Add embedded web server for tests ...................................................................... Add embedded web server for tests Change-Id: I2d3815b2dfe912ea56a23cff3c4a3c7863b1502b Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M README.adoc M sdk/spec/clusters_kerberos_service_spec.rb M sdk/spec/clusters_service_spec.rb A sdk/spec/connection_create_spec.rb D sdk/spec/connection_spec.rb A sdk/spec/connection_timeout_spec.rb A sdk/spec/connection_use_spec.rb M sdk/spec/data_centers_service_spec.rb D sdk/spec/parameters.yaml M sdk/spec/spec_helper.rb M sdk/spec/storage_domains_service_spec.rb M sdk/spec/vms_service_spec.rb 12 files changed, 512 insertions(+), 470 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/54444 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2d3815b2dfe912ea56a23cff3c4a3c7863b1502b Gerrit-PatchSet: 5 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-sdk-ruby[master]: Make it clear that the "sso" parameters are optional
by juan.hernandez＠redhat.com 18 Mar '16

18 Mar '16

Juan Hernandez has submitted this change and it was merged. Change subject: Make it clear that the "sso" parameters are optional ...................................................................... Make it clear that the "sso" parameters are optional This patch improves the documentation of the constructor of the connection, trying to make it clear that the use of the "sso" parameters is optional, and not needed in most usage scenarios. Change-Id: I518e11391fdc37de1a82881bf06bd8613a8cb535 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M sdk/lib/ovirtsdk4/http.rb 1 file changed, 19 insertions(+), 6 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/54922 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I518e11391fdc37de1a82881bf06bd8613a8cb535 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 jenkins[master]: Using the image std ci template to have less artifacts
by dcaro＠redhat.com 18 Mar '16

18 Mar '16

David Caro has submitted this change and it was merged. Change subject: Using the image std ci template to have less artifacts ...................................................................... Using the image std ci template to have less artifacts Change-Id: Iad5a41c20f1dee48e5c1ec8d2a897291812efda9 Signed-off-by: Tolik Litovsky <tlitovsk(a)redhat.com> Signed-off-by: David Caro <dcaroest(a)redhat.com> --- M jobs/confs/projects/ovirt-node-ng/ovirt-node-ng.yaml M jobs/confs/yaml/templates/standard-stage.yaml 2 files changed, 17 insertions(+), 8 deletions(-) Approvals: Fabian Deutsch: Looks good to me Jenkins CI: Passed CI tests David Caro: Verified; Ready for merge -- To view, visit https://gerrit.ovirt.org/54926 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iad5a41c20f1dee48e5c1ec8d2a897291812efda9 Gerrit-PatchSet: 4 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Fabian Deutsch <fabiand(a)redhat.com> Gerrit-Reviewer: Anatoly Litovsky <tlitovsk(a)redhat.com> Gerrit-Reviewer: David Caro <dcaro(a)redhat.com> Gerrit-Reviewer: Fabian Deutsch <fabiand(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0