February 2014 - Engine-commits - Ovirt List Archives

Change in ovirt-engine[master]: tools: Moves temp file creation from .sh to Java in engine-m...

by yzaslavs＠redhat.com

Yair Zaslavsky has submitted this change and it was merged. Change subject: tools: Moves temp file creation from .sh to Java in engine-manage-domains ...................................................................... tools: Moves temp file creation from .sh to Java in engine-manage-domains Moves creation of engine-config custom properties temporary file from engine-manage-domains.sh to Java, so -propertiesFile command line argument can be removed. Change-Id: Ia6fd2c5066e0e0cf12d10be32ec6c36a8a91bfaf Bug-Url: https://bugzilla.redhat.com/904029 Signed-off-by: Martin Perina <mperina(a)redhat.com> --- M backend/manager/tools/src/main/java/org/ovirt/engine/core/domains/ManageDomains.java M backend/manager/tools/src/main/java/org/ovirt/engine/core/domains/ManageDomainsResultEnum.java M packaging/bin/engine-manage-domains.sh 3 files changed, 89 insertions(+), 68 deletions(-) Approvals: Martin Peřina: Verified Yair Zaslavsky: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/23840 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia6fd2c5066e0e0cf12d10be32ec6c36a8a91bfaf Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

10 years, 10 months

1
0
0 / 0

Change in ovirt-engine[master]: tools: Removes inclusion of config file in engine-manage-dom...

by yzaslavs＠redhat.com

Yair Zaslavsky has submitted this change and it was merged. Change subject: tools: Removes inclusion of config file in engine-manage-domains.sh ...................................................................... tools: Removes inclusion of config file in engine-manage-domains.sh Removes unneeded inclusion of engine-manage-domains.conf in engine-manage-domains.sh. The variables contained in this file are needed later in java code, but not in engine-manage-domains.sh. Change-Id: Ia26468cbcc64c44adb9805c258f1625c574b4ab5 Bug-Url: https://bugzilla.redhat.com/904029 Signed-off-by: Martin Perina <mperina(a)redhat.com> --- M packaging/bin/engine-manage-domains.sh 1 file changed, 0 insertions(+), 16 deletions(-) Approvals: Martin Peřina: Verified Yair Zaslavsky: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/23839 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia26468cbcc64c44adb9805c258f1625c574b4ab5 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com>

10 years, 10 months

1
0
0 / 0

Change in ovirt-engine[master]: tools: Removes -interactive option from engine-manage-domains

by yzaslavs＠redhat.com

Yair Zaslavsky has submitted this change and it was merged. Change subject: tools: Removes -interactive option from engine-manage-domains ...................................................................... tools: Removes -interactive option from engine-manage-domains Removes -interactive option from engine-manage-domains. If -passwordFile option is not set, then the password will be read interactively. Change-Id: I9bd764a50f6e6d237bc65d6e7a72ac2cc636786a Bug-Url: https://bugzilla.redhat.com/904029 Signed-off-by: Martin Perina <mperina(a)redhat.com> --- M backend/manager/tools/src/main/java/org/ovirt/engine/core/domains/ManageDomains.java M packaging/bin/engine-manage-domains.sh M packaging/man/man8/engine-manage-domains.8 3 files changed, 5 insertions(+), 13 deletions(-) Approvals: Martin Peřina: Verified Yair Zaslavsky: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/23838 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I9bd764a50f6e6d237bc65d6e7a72ac2cc636786a Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

10 years, 10 months

1
0
0 / 0

Change in ovirt-engine[ovirt-engine-3.3]: engine: Add config value for Gluster task support

by iheim＠redhat.com

Itamar Heim has submitted this change and it was merged. Change subject: engine: Add config value for Gluster task support ...................................................................... engine: Add config value for Gluster task support Added database config value for gluster task support. Change-Id: I9a62b558a91c9b52b6a8d7a892e0420f2d1ec51e Signed-off-by: Sahina Bose <sabose(a)redhat.com> --- M packaging/dbscripts/upgrade/pre_upgrade/0000_config.sql 1 file changed, 4 insertions(+), 0 deletions(-) Approvals: Itamar Heim: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/23982 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I9a62b558a91c9b52b6a8d7a892e0420f2d1ec51e Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.3 Gerrit-Owner: Sahina Bose <sabose(a)redhat.com> Gerrit-Reviewer: Itamar Heim <iheim(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

10 years, 10 months

1
0
0 / 0

Change in ovirt-engine[ovirt-engine-3.4]: webadmin: AddRemoveRowWidget to only add ghost entry if no i...

by iheim＠redhat.com

Itamar Heim has submitted this change and it was merged. Change subject: webadmin: AddRemoveRowWidget to only add ghost entry if no items ...................................................................... webadmin: AddRemoveRowWidget to only add ghost entry if no items If there are items in the passed collection, then there's no need to add a ghost entry by default, but only have a plus button next to the last entry. Change-Id: I120e9183d739213b3a26227cc8c3b738ee3b54dc Bug-Url: https://bugzilla.redhat.com/1020861 Bug-Url: https://bugzilla.redhat.com/1058205 Signed-off-by: Lior Vernia <lvernia(a)redhat.com> --- M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/AddRemoveRowWidget.java M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/form/key_value/KeyValueWidget.java 2 files changed, 25 insertions(+), 15 deletions(-) Approvals: Tomas Jelinek: Looks good to me, approved Lior Vernia: Verified -- To view, visit http://gerrit.ovirt.org/23826 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I120e9183d739213b3a26227cc8c3b738ee3b54dc Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.4 Gerrit-Owner: Lior Vernia <lvernia(a)redhat.com> Gerrit-Reviewer: Itamar Heim <iheim(a)redhat.com> Gerrit-Reviewer: Lior Vernia <lvernia(a)redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

10 years, 10 months

1
0
0 / 0

Change in ovirt-engine[ovirt-engine-3.4]: core: Add tests to StorageHandlingCommandBaseTest

by iheim＠redhat.com

Itamar Heim has submitted this change and it was merged. Change subject: core: Add tests to StorageHandlingCommandBaseTest ...................................................................... core: Add tests to StorageHandlingCommandBaseTest Since the removal of DC type it's important to have test to test the logic of whether attachment of domains to a pool is allowed, this patch adds tests for the StorageHandlingCommandBase which holds most of the logic that determines attachment is allowed Change-Id: I97e45957f7ff0fe4c9e7d88e0d8dbab7e874c79b Signed-off-by: Tal Nisan <tnisan(a)redhat.com> Relates-To: https://bugzilla.redhat.com/1038053 --- M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/CanDoActionTestUtils.java M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/storage/StorageHandlingCommandBaseTest.java 2 files changed, 212 insertions(+), 18 deletions(-) Approvals: Tal Nisan: Verified Allon Mureinik: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/23960 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I97e45957f7ff0fe4c9e7d88e0d8dbab7e874c79b Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.4 Gerrit-Owner: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Itamar Heim <iheim(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

10 years, 10 months

1
0
0 / 0

Change in ovirt-engine[ovirt-engine-3.4]: packaging: setup: distro-rpm: fix typo in new package detection

by iheim＠redhat.com

Itamar Heim has submitted this change and it was merged. Change subject: packaging: setup: distro-rpm: fix typo in new package detection ...................................................................... packaging: setup: distro-rpm: fix typo in new package detection Change-Id: Ibb2aa3ce1f1d9f0d4c5b250e0f70cbe22e931b71 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M packaging/setup/plugins/ovirt-engine-setup/ovirt-engine-common/distro-rpm/packages.py 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Alon Bar-Lev: Verified Sandro Bonazzola: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/23986 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ibb2aa3ce1f1d9f0d4c5b250e0f70cbe22e931b71 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.4 Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Itamar Heim <iheim(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>

10 years, 10 months

1
0
0 / 0

Change in ovirt-engine[ovirt-engine-3.4]: webadmin: Filter storage domains of different type in live s...

by iheim＠redhat.com

Itamar Heim has submitted this change and it was merged. Change subject: webadmin: Filter storage domains of different type in live storage migration ...................................................................... webadmin: Filter storage domains of different type in live storage migration This change filters the destination domain by the type of the source domain when performing live storage migration, if the source domain is a file domain, the destination domains that will appear will be only file domains and vice versa Change-Id: I94f9feb9eb2b678e76653bed0b022ed05af40fc1 Signed-off-by: Tal Nisan <tnisan(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/storage/MoveOrCopyDiskModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/MoveDiskModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/VmDiskListModel.java M frontend/webadmin/modules/uicompat/src/main/java/org/ovirt/engine/ui/uicompat/UIConstants.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/storage/DisksAllocationPopupView.ui.xml 5 files changed, 49 insertions(+), 17 deletions(-) Approvals: Tal Nisan: Verified Daniel Erez: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/23963 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I94f9feb9eb2b678e76653bed0b022ed05af40fc1 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.4 Gerrit-Owner: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Itamar Heim <iheim(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

10 years, 10 months

1
0
0 / 0

Change in ovirt-engine[ovirt-engine-3.4]: core: Block live storage migration to domain of a different ...

by iheim＠redhat.com

Itamar Heim has submitted this change and it was merged. Change subject: core: Block live storage migration to domain of a different type ...................................................................... core: Block live storage migration to domain of a different type This change fails live storage migration CDA part in case that the source storage domain and the destination domain are not both either a file type domain or a block domain Change-Id: Iec4c34132a1a619527627cfa47fa99df3a023b62 Signed-off-by: Tal Nisan <tnisan(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/lsm/LiveMigrateVmDisksCommand.java M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/lsm/LiveMigrateVmDisksCommandTest.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/StorageType.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/errors/VdcBllMessages.java M backend/manager/modules/dal/src/main/resources/bundles/AppErrors.properties M frontend/webadmin/modules/frontend/src/main/java/org/ovirt/engine/ui/frontend/AppErrors.java M frontend/webadmin/modules/userportal-gwtp/src/main/resources/org/ovirt/engine/ui/frontend/AppErrors.properties M frontend/webadmin/modules/webadmin/src/main/resources/org/ovirt/engine/ui/frontend/AppErrors.properties 8 files changed, 60 insertions(+), 14 deletions(-) Approvals: Itamar Heim: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/23962 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iec4c34132a1a619527627cfa47fa99df3a023b62 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.4 Gerrit-Owner: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Itamar Heim <iheim(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

10 years, 10 months

1
0
0 / 0

Change in ovirt-log-collector[ovirt-log-collector-3.3]: sos: engine: collect full service configuration

by sbonazzo＠redhat.com

Sandro Bonazzola has submitted this change and it was merged. Change subject: sos: engine: collect full service configuration ...................................................................... sos: engine: collect full service configuration Collect also /var/tmp/ovirt-engine/config content. Change-Id: Id41c3859731a77cb2c771d327730c277b81a8c54 Bug-Url: https://bugzilla.redhat.com/1060673 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> (cherry picked from commit 4f09866ebf9afd3462d02a0e1d349093bae592b9) (cherry picked from commit 512f17198bb5625c931a9dd98b612f7b437fe63a) --- M src/sos/plugins/engine.py 1 file changed, 13 insertions(+), 0 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/23989 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id41c3859731a77cb2c771d327730c277b81a8c54 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-log-collector Gerrit-Branch: ovirt-log-collector-3.3 Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Kiril Nesenko <knesenko(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com>

10 years, 10 months

1
0
0 / 0

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

Engine-commits February 2014