June 2014 - Engine-commits - oVirt List Archives

Change in jenkins[master]: Added unboundid-ldapsdk jobs
by dcaroest＠redhat.com 27 Jun '14

27 Jun '14

David Caro has submitted this change and it was merged. Change subject: Added unboundid-ldapsdk jobs ...................................................................... Added unboundid-ldapsdk jobs That include releng based scripts and templates Change-Id: Id236668a2312a8205b8b7bbaafa775f316cc30df Signed-off-by: David Caro <dcaroest(a)redhat.com> --- A jobs/confs/shell-scripts/build_local_srcrpm.sh A jobs/confs/shell-scripts/build_local_srcrpm_releng.sh C jobs/confs/shell-scripts/build_mock_srcrpm_releng.sh R jobs/confs/shell-scripts/mock_build_onlyrpm.sh M jobs/confs/yaml/builders/mock_install.yaml M jobs/confs/yaml/builders/rpm_mock.yaml A jobs/confs/yaml/jobs/unboundid-ldapsdk/unboundid-ldapsdk_create-rpms.yaml A jobs/confs/yaml/scms/releng-tools.yaml M jobs/confs/yaml/templates/create-rpms_mock.yaml 9 files changed, 200 insertions(+), 79 deletions(-) Approvals: David Caro: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/29296 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id236668a2312a8205b8b7bbaafa775f316cc30df Gerrit-PatchSet: 2 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: David Caro <dcaroest(a)redhat.com> Gerrit-Reviewer: David Caro <dcaroest(a)redhat.com> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Kiril Nesenko <knesenko(a)redhat.com> Gerrit-Reviewer: Ohad Basan <obasan(a)redhat.com> Gerrit-Reviewer: Paz Dangur <pdangur(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in jenkins[master]: Fixed mock run on cross arch environments
by dcaroest＠redhat.com 27 Jun '14

27 Jun '14

David Caro has submitted this change and it was merged. Change subject: Fixed mock run on cross arch environments ...................................................................... Fixed mock run on cross arch environments It seems that removing the __db files was not enough, this patch makes sure it works everywhere. Change-Id: I78401d45b08bceef4360db95573586ced01e12d9 Signed-off-by: David Caro <dcaroest(a)redhat.com> --- M jobs/confs/shell-scripts/mock_install.sh 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: David Caro: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/29298 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I78401d45b08bceef4360db95573586ced01e12d9 Gerrit-PatchSet: 1 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: David Caro <dcaroest(a)redhat.com> Gerrit-Reviewer: David Caro <dcaroest(a)redhat.com> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Kiril Nesenko <knesenko(a)redhat.com> Gerrit-Reviewer: Ohad Basan <obasan(a)redhat.com> Gerrit-Reviewer: Paz Dangur <pdangur(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: core: Moving out IrsProxyData to a separate file.
by yzaslavs＠redhat.com 27 Jun '14

27 Jun '14

Yair Zaslavsky has submitted this change and it was merged. Change subject: core: Moving out IrsProxyData to a separate file. ...................................................................... core: Moving out IrsProxyData to a separate file. This will make the code of IrsBrokerCommand be more clear, and easy to be adjusted, when needed. Change-Id: Ie0080ebb2443766a884dd9d9a1082b774a61e87b Signed-off-by: Yair Zaslavsky <yzaslavs(a)redhat.com> --- M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/irsbroker/IrsBrokerCommand.java A backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/irsbroker/IrsProxyData.java 2 files changed, 1,500 insertions(+), 1,472 deletions(-) Approvals: Yair Zaslavsky: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/29264 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie0080ebb2443766a884dd9d9a1082b774a61e87b Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Liron Aravot <laravot(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: core: Modify constructor to be compatible with caller method
by yzaslavs＠redhat.com 27 Jun '14

27 Jun '14

Yair Zaslavsky has submitted this change and it was merged. Change subject: core: Modify constructor to be compatible with caller method ...................................................................... core: Modify constructor to be compatible with caller method CreateOvfStoresForStorageDomainCommand is being called without context although its constructor is only supported with context parameter, so we will always get runtime exception when the command is being called. The proposed fix remove the context from the CreateOvfStoresForStorageDomainCommand constructor Change-Id: Ib2cd44301ccf992541ecd0f9572fd7afde4a123d Signed-off-by: Maor Lipchuk <mlipchuk(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/CreateOvfStoresForStorageDomainCommand.java 1 file changed, 2 insertions(+), 4 deletions(-) Approvals: Yair Zaslavsky: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/29315 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib2cd44301ccf992541ecd0f9572fd7afde4a123d Gerrit-PatchSet: 1 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: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: core: Modify constructor to be compatible with caller
by yzaslavs＠redhat.com 27 Jun '14

27 Jun '14

Yair Zaslavsky has submitted this change and it was merged. Change subject: core: Modify constructor to be compatible with caller ...................................................................... core: Modify constructor to be compatible with caller ProcessOvfUpdateForStorageDomainCommand is being called without context, although the constructor is only supported with context parameter, so we will always get runtime exception when being called. The proposed fix remove the context from the ProcessOvfUpdateForStorageDomainCommand constructor Change-Id: Id4c1c7f36876b51d2194e69b95a3d7c20f43b659 Signed-off-by: Maor Lipchuk <mlipchuk(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ProcessOvfUpdateForStorageDomainCommand.java 1 file changed, 2 insertions(+), 4 deletions(-) Approvals: Maor Lipchuk: Verified Yair Zaslavsky: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/29314 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id4c1c7f36876b51d2194e69b95a3d7c20f43b659 Gerrit-PatchSet: 2 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: Liron Aravot <laravot(a)redhat.com> Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: webadmin: enable import data domains on storage dialog
by derez＠redhat.com 26 Jun '14

26 Jun '14

Daniel Erez has submitted this change and it was merged. Change subject: webadmin: enable import data domains on storage dialog ...................................................................... webadmin: enable import data domains on storage dialog Import storage domain dialog: adding File/Block data domains to type list. Change-Id: I8fab33f28995a59849341541d85b6758de82a34b Signed-off-by: Daniel Erez <derez(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/dataprovider/AsyncDataProvider.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/storage/ImportStorageModelBehavior.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/storage/StoragePopupView.java 3 files changed, 90 insertions(+), 10 deletions(-) Approvals: Daniel Erez: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/28833 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8fab33f28995a59849341541d85b6758de82a34b Gerrit-PatchSet: 17 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: webadmin: import FCP data storage domain dialog
by derez＠redhat.com 26 Jun '14

26 Jun '14

Daniel Erez has submitted this change and it was merged. Change subject: webadmin: import FCP data storage domain dialog ...................................................................... webadmin: import FCP data storage domain dialog Import domain dialog -> added a view for import FCP data domain which contains a panel for storage domains that are candidate for import. Feature page: http://www.ovirt.org/Features/ImportStorageDomain#GUI_Perspective_2 Change-Id: I529680fbf0180874ffad02a050601fd1294fbec1 Signed-off-by: Daniel Erez <derez(a)redhat.com> --- A frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/storage/ImportFcpStorageView.java A frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/storage/ImportFcpStorageView.ui.xml A frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/storage/ImportFcpStorageModel.java 3 files changed, 199 insertions(+), 0 deletions(-) Approvals: Tal Nisan: Looks good to me, approved Daniel Erez: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/29166 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I529680fbf0180874ffad02a050601fd1294fbec1 Gerrit-PatchSet: 14 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: webadmin: import iSCSI data storage domain dialog
by derez＠redhat.com 26 Jun '14

26 Jun '14

Daniel Erez has submitted this change and it was merged. Change subject: webadmin: import iSCSI data storage domain dialog ...................................................................... webadmin: import iSCSI data storage domain dialog Import domain dialog -> added a view for import iSCSI data domain which consists of two panels: * Target discovery. * Storage domains that are candidate for import. Feature page: http://www.ovirt.org/Features/ImportStorageDomain#GUI_Perspective_2 Change-Id: If379f5e41544fda90cb3e31e38784b768c994e5c Signed-off-by: Daniel Erez <derez(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/ValidatedPanelWidget.java M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/ValidatedPanelWidget.ui.xml M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/storage/AbstractStorageView.java A frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/storage/ImportIscsiStorageView.java A frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/storage/ImportIscsiStorageView.ui.xml M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/storage/IscsiDiscoverTargetsView.java M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/storage/IscsiDiscoverTargetsView.ui.xml A frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/storage/ImportIscsiStorageModel.java A frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/storage/ImportSanStorageModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/storage/ImportStorageModelBehavior.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/storage/IscsiStorageModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/storage/NewEditStorageModelBehavior.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/storage/SanStorageModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/storage/SanStorageModelBase.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/storage/SanTargetModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/storage/StorageListModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/storage/StorageModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/storage/StorageModelBehavior.java M frontend/webadmin/modules/uicompat/src/main/java/org/ovirt/engine/ui/uicompat/UIConstants.java 20 files changed, 741 insertions(+), 44 deletions(-) Approvals: Daniel Erez: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/29173 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If379f5e41544fda90cb3e31e38784b768c994e5c Gerrit-PatchSet: 13 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: core: introduce AddExistingBlockStorageDomain command
by derez＠redhat.com 26 Jun '14

26 Jun '14

Daniel Erez has submitted this change and it was merged. Change subject: core: introduce AddExistingBlockStorageDomain command ...................................................................... core: introduce AddExistingBlockStorageDomain command Introduce a new command for importing data block (iSCSI/FCP) domains. Change-Id: Icdebb7d390d14e88d84125dd412d938dacb72a2b Signed-off-by: Daniel Erez <derez(a)redhat.com> --- A backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/AddExistingBlockStorageDomainCommand.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/action/VdcActionType.java 2 files changed, 80 insertions(+), 0 deletions(-) Approvals: Daniel Erez: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/29301 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Icdebb7d390d14e88d84125dd412d938dacb72a2b Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: core: introduce GetUnregisteredBlockStorageDomains query
by derez＠redhat.com 26 Jun '14

26 Jun '14

Daniel Erez has submitted this change and it was merged. Change subject: core: introduce GetUnregisteredBlockStorageDomains query ...................................................................... core: introduce GetUnregisteredBlockStorageDomains query Introduced a new command for discovering unregistered block (iSCSI/FCP) storage domains (of type Data). The query should get as parameters the host ID, StorageType (iSCSI/FCP) and a list of StorageServerConnections (applicable only for iSCSI). Finding unregistered domains flow is as follows: * Connect into the specified targets (using ConnectStorageToVds). * Retrieve associated LUNs (using GetDeviceList). * Filter irrelevant LUNs by crossing with the specified targets and LUNs that are already a part of registered storage domains. * Fetching volume group IDs from the LUNs list. * For each VG-ID, use GetVGInfo to get VG-Name (storage domains ID). * Invoke GetStorageDomainInfo on each storage domain ID and build the StorageDomain objects. * Finally, return to the client a list of storage domains and successfully logged-in connections. Change-Id: I71cad5562f8d7dcb39c5a99b7daea28ccb4a00cf Signed-off-by: Daniel Erez <derez(a)redhat.com> --- A backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/GetUnregisteredBlockStorageDomainsQuery.java A backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/storage/GetUnregisteredBlockStorageDomainsQueryTest.java A backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/queries/GetUnregisteredBlockStorageDomainsParameters.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/queries/VdcQueryType.java 4 files changed, 606 insertions(+), 0 deletions(-) Approvals: Daniel Erez: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/29172 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I71cad5562f8d7dcb39c5a99b7daea28ccb4a00cf Gerrit-PatchSet: 10 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0