Change in ovirt-engine[master]: core: Call end action once CopyTaskHandler is finished.
by mlipchuk@redhat.com
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>
8 years, 10 months
Change in ovirt-engine[master]: webadmin: Warning for unsupported memory restore (Custom Sna...
by ahadas@redhat.com
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>
8 years, 10 months
Change in ovirt-engine[master]: Revert "core: Migration bandwidth columns added to stored pr...
by ahadas@redhat.com
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>
8 years, 10 months
Change in ovirt-engine[ovirt-engine-3.6]: webadmin: Check box tooltip
by tnisan@redhat.com
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>
8 years, 10 months
Change in ovirt-engine[master]: core: Migration bandwidth columns added to stored procedures
by ahadas@redhat.com
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>
8 years, 10 months
Change in jenkins[master]: Remove all the log gathering from the system job
by dcaro@redhat.com
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>
8 years, 10 months
Change in ovirt-engine-sdk-ruby[master]: Remove the "kerberos" test tag
by juan.hernandez@redhat.com
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>
8 years, 10 months
Change in ovirt-engine-sdk-ruby[master]: Add embedded web server for tests
by juan.hernandez@redhat.com
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>
8 years, 10 months
Change in ovirt-engine-sdk-ruby[master]: Make it clear that the "sso" parameters are optional
by juan.hernandez@redhat.com
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>
8 years, 10 months
Change in jenkins[master]: Using the image std ci template to have less artifacts
by dcaro@redhat.com
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>
8 years, 10 months