March 2014 - Engine-commits - oVirt List Archives

Change in ovirt-reports[master]: reports: edited ic query-only_hosts_hosts_list
by ydary＠redhat.com 17 Mar '14

17 Mar '14

Yaniv Dary has submitted this change and it was merged. Change subject: reports: edited ic query-only_hosts_hosts_list ...................................................................... reports: edited ic query-only_hosts_hosts_list I edited the, only_hosts_hosts_list, input control query, so it will be more readable and added comments. Change-Id: I002de9e97065f9c9be961a22218cc677874951a4 Signed-off-by: Shirly Radco <sradco(a)redhat.com> --- M packaging/ovirt-reports/resources/reports_resources/JDBC/Input_Controls/Regular_Input_Controls/Only_Hosts/P_Host_ListID.xml 1 file changed, 94 insertions(+), 33 deletions(-) Approvals: Shirly Radco: Verified; Looks good to me, but someone else must approve Yaniv Dary: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/25418 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I002de9e97065f9c9be961a22218cc677874951a4 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-reports Gerrit-Branch: master Gerrit-Owner: Shirly Radco <sradco(a)redhat.com> Gerrit-Reviewer: Shirly Radco <sradco(a)redhat.com> Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com>

1 0

Change in ovirt-reports[master]: reports: edited ic query-only_hosts_host_type
by ydary＠redhat.com 17 Mar '14

17 Mar '14

Yaniv Dary has submitted this change and it was merged. Change subject: reports: edited ic query-only_hosts_host_type ...................................................................... reports: edited ic query-only_hosts_host_type I edited the, only_hosts_host_type, input control query, so it will be more readable and added comments. Change-Id: Ic42e9313c3ab8ebd66e8f6023343f343e345497d Signed-off-by: Shirly Radco <sradco(a)redhat.com> --- M packaging/ovirt-reports/resources/reports_resources/JDBC/Input_Controls/Regular_Input_Controls/Only_Hosts/P_Host_Type.xml 1 file changed, 92 insertions(+), 30 deletions(-) Approvals: Shirly Radco: Verified; Looks good to me, but someone else must approve Yaniv Dary: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/25414 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic42e9313c3ab8ebd66e8f6023343f343e345497d Gerrit-PatchSet: 5 Gerrit-Project: ovirt-reports Gerrit-Branch: master Gerrit-Owner: Shirly Radco <sradco(a)redhat.com> Gerrit-Reviewer: Shirly Radco <sradco(a)redhat.com> Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com>

1 0

Change in ovirt-reports[master]: reports: edited ic query-only_hosts_dc
by ydary＠redhat.com 17 Mar '14

17 Mar '14

Yaniv Dary has submitted this change and it was merged. Change subject: reports: edited ic query-only_hosts_dc ...................................................................... reports: edited ic query-only_hosts_dc I edited the, only_hosts_dc, input control query, so it will be more readable and added comments. Change-Id: I8505785fc213eb91708c27f20b31a60703d67079 Signed-off-by: Shirly Radco <sradco(a)redhat.com> --- M packaging/ovirt-reports/resources/reports_resources/JDBC/Input_Controls/Regular_Input_Controls/Only_Hosts/P_DataCenter_ID.xml 1 file changed, 129 insertions(+), 42 deletions(-) Approvals: Shirly Radco: Verified; Looks good to me, but someone else must approve Yaniv Dary: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/25369 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8505785fc213eb91708c27f20b31a60703d67079 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-reports Gerrit-Branch: master Gerrit-Owner: Shirly Radco <sradco(a)redhat.com> Gerrit-Reviewer: Shirly Radco <sradco(a)redhat.com> Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com>

1 0

Change in ovirt-reports[master]: reports: edited ic query-only_hosts_cluster
by ydary＠redhat.com 16 Mar '14

16 Mar '14

Yaniv Dary has submitted this change and it was merged. Change subject: reports: edited ic query-only_hosts_cluster ...................................................................... reports: edited ic query-only_hosts_cluster I edited the, only_hosts_cluster, input control query, so it will be more readable and added comments. Change-Id: Ia894e13bed179a6df8ac170f37bd6ebaa7238c23 Signed-off-by: Shirly Radco <sradco(a)redhat.com> --- M packaging/ovirt-reports/resources/reports_resources/JDBC/Input_Controls/Regular_Input_Controls/Only_Hosts/P_Cluster_ID.xml 1 file changed, 111 insertions(+), 32 deletions(-) Approvals: Yaniv Dary: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/25318 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia894e13bed179a6df8ac170f37bd6ebaa7238c23 Gerrit-PatchSet: 6 Gerrit-Project: ovirt-reports Gerrit-Branch: master Gerrit-Owner: Shirly Radco <sradco(a)redhat.com> Gerrit-Reviewer: Shirly Radco <sradco(a)redhat.com> Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com>

1 0

Change in ovirt-reports[master]: reports: edited ic query-vm_type
by ydary＠redhat.com 16 Mar '14

16 Mar '14

Yaniv Dary has submitted this change and it was merged. Change subject: reports: edited ic query-vm_type ...................................................................... reports: edited ic query-vm_type I edited the, vm_type, input control query, so it will be more readable and added comments. Change-Id: I1c31944cda87b94376c19269d6b08a09e1794afb Signed-off-by: Shirly Radco <sradco(a)redhat.com> --- M packaging/ovirt-reports/resources/reports_resources/JDBC/Input_Controls/Period_Affected/single_select/vm_ic/P_VM_Type.xml 1 file changed, 96 insertions(+), 90 deletions(-) Approvals: Shirly Radco: Verified; Looks good to me, but someone else must approve Yaniv Dary: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/25311 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1c31944cda87b94376c19269d6b08a09e1794afb Gerrit-PatchSet: 4 Gerrit-Project: ovirt-reports Gerrit-Branch: master Gerrit-Owner: Shirly Radco <sradco(a)redhat.com> Gerrit-Reviewer: Shirly Radco <sradco(a)redhat.com> Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com>

1 0

Change in ovirt-reports[master]: reports: edited ic query-storage_domain_type
by ydary＠redhat.com 16 Mar '14

16 Mar '14

Yaniv Dary has submitted this change and it was merged. Change subject: reports: edited ic query-storage_domain_type ...................................................................... reports: edited ic query-storage_domain_type I edited the, storage_domain_type, input control query, so it will be more readable and added comments. Change-Id: Ic79279df2b59592a8ca93517d52b7c7f2ff42e6c Signed-off-by: Shirly Radco <sradco(a)redhat.com> --- M packaging/ovirt-reports/resources/reports_resources/JDBC/Input_Controls/Period_Affected/single_select/storage_ic/P_Storage_Type.xml 1 file changed, 127 insertions(+), 44 deletions(-) Approvals: Shirly Radco: Verified; Looks good to me, but someone else must approve Yaniv Dary: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/25299 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic79279df2b59592a8ca93517d52b7c7f2ff42e6c Gerrit-PatchSet: 8 Gerrit-Project: ovirt-reports Gerrit-Branch: master Gerrit-Owner: Shirly Radco <sradco(a)redhat.com> Gerrit-Reviewer: Shirly Radco <sradco(a)redhat.com> Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com>

1 0

Change in ovirt-engine[ovirt-engine-3.4]: core: export & import highly available parameters
by iheim＠redhat.com 16 Mar '14

16 Mar '14

Itamar Heim has submitted this change and it was merged. Change subject: core: export & import highly available parameters ...................................................................... core: export & import highly available parameters The 'auto startup' and 'priority' attributes of VM/Template were not copied on export & import operations. Now they are. Change-Id: I194b70181cf4ab0420044a22dd8a382766d9019d Bug-Url: https://bugzilla.redhat.com/1057048 Signed-off-by: Arik Hadas <ahadas(a)redhat.com> --- M backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/ovf/OvfProperties.java M backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/ovf/OvfReader.java M backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/ovf/OvfVmReader.java M backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/ovf/OvfVmWriter.java M backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/ovf/OvfWriter.java 5 files changed, 21 insertions(+), 9 deletions(-) Approvals: Omer Frenkel: Looks good to me, approved Arik Hadas: Verified -- To view, visit http://gerrit.ovirt.org/25765 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I194b70181cf4ab0420044a22dd8a382766d9019d Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.4 Gerrit-Owner: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Itamar Heim <iheim(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[ovirt-engine-3.4]: core: Fix KeystoneAuthUrl example's description
by iheim＠redhat.com 16 Mar '14

16 Mar '14

Itamar Heim has submitted this change and it was merged. Change subject: core: Fix KeystoneAuthUrl example's description ...................................................................... core: Fix KeystoneAuthUrl example's description The example of KeystoneAuthUrl should include the version. Change-Id: I387a15e66e8a45b52c9189253ac7b5898e9b2a75 Bug-Url: https://bugzilla.redhat.com/1075997 Signed-off-by: Moti Asayag <masayag(a)redhat.com> --- M packaging/etc/engine-config/engine-config.properties 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Mike Kolesnik: Looks good to me, approved Moti Asayag: Verified -- To view, visit http://gerrit.ovirt.org/25770 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I387a15e66e8a45b52c9189253ac7b5898e9b2a75 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.4 Gerrit-Owner: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Itamar Heim <iheim(a)redhat.com> Gerrit-Reviewer: Mike Kolesnik <mkolesni(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[ovirt-engine-3.4]: restapi : Missing opertions for snapshots in RESTAPI
by iheim＠redhat.com 16 Mar '14

16 Mar '14

Itamar Heim has submitted this change and it was merged. Change subject: restapi : Missing opertions for snapshots in RESTAPI ...................................................................... restapi : Missing opertions for snapshots in RESTAPI In RESTAPI some opertions for snapshot are missing which are available in webadmin. This patch adds preview, commit and undo operations to the snapshot resource. Change-Id: If8c6d25daf7e716a7c857295b6ceb1741cbaf7f7 Bug-Url: https://bugzilla.redhat.com/867339 Signed-off-by: Maor Lipchuk <mlipchuk(a)redhat.com> --- M backend/manager/modules/restapi/interface/definition/src/main/java/org/ovirt/engine/api/resource/VmResource.java M backend/manager/modules/restapi/interface/definition/src/main/resources/api.xsd M backend/manager/modules/restapi/interface/definition/src/main/resources/rsdl_metadata.yaml M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendVmResource.java M backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/BackendVmResourceTest.java 5 files changed, 130 insertions(+), 1 deletion(-) Approvals: Daniel Erez: Looks good to me, approved Maor Lipchuk: Verified -- To view, visit http://gerrit.ovirt.org/25691 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If8c6d25daf7e716a7c857295b6ceb1741cbaf7f7 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.4 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: Itamar Heim <iheim(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: Ori Liel <oliel(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[ovirt-engine-3.4]: engine: Add action type for snapshot.
by iheim＠redhat.com 16 Mar '14

16 Mar '14

Itamar Heim has submitted this change and it was merged. Change subject: engine: Add action type for snapshot. ...................................................................... engine: Add action type for snapshot. The following patch changes the API between the client and the engine when preview/commit or undo a snapshot. The patch introduces the following changes: * Adding an enum which distinguishes between several operations on a snapshot: Undo Commit Restore From Stateless status * Use of snapshot instead snapshotId * Changing the client to call to the engine with the operation on snapshot * Removing snapshotId from the constructor of the parameters, since the snapshot is only on a VM, and there is no reason to transfer it, but better that the engine will fetch it, it self. The reason for this change, is for REST operations to be implemented in the following patches Change-Id:If877befc5058c3412ae3d3731d5beacbc09e5c97 Bug-Url: https://bugzilla.redhat.com/867339 Signed-off-by: Maor Lipchuk <mlipchuk(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RestoreAllSnapshotsCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RestoreStatelessVmCommand.java M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/RestoreAllSnapshotCommandTest.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/action/RestoreAllSnapshotsParameters.java A backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/SnapshotActionEnum.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendSnapshotResource.java M backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/BackendSnapshotResourceTest.java M frontend/webadmin/modules/gwt-common/src/main/resources/org/ovirt/engine/core/Common.gwt.xml M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/VmSnapshotListModel.java 9 files changed, 99 insertions(+), 72 deletions(-) Approvals: Daniel Erez: Looks good to me, approved Maor Lipchuk: Verified -- To view, visit http://gerrit.ovirt.org/25690 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If877befc5058c3412ae3d3731d5beacbc09e5c97 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.4 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: Itamar Heim <iheim(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