March 2013 - Engine-commits - oVirt List Archives

Change in ovirt-iso-uploader[ovirt-iso-uploader-3.2]: Fixing format_ssh_command
by knesenko＠redhat.com 31 Mar '13

31 Mar '13

Kiril Nesenko has submitted this change and it was merged. Change subject: Fixing format_ssh_command ...................................................................... Fixing format_ssh_command format_ssh_command failed setting the Identity file causing an error like no such file or directory. Change-Id: Ic401993a4ebbcfa99335673e70fc0ae6d191b279 Bug-Url: https://bugzilla.redhat.com/904719 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M src/__main__.py 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Kiril Nesenko: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/13458 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic401993a4ebbcfa99335673e70fc0ae6d191b279 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-iso-uploader Gerrit-Branch: ovirt-iso-uploader-3.2 Gerrit-Owner: Kiril Nesenko <knesenko(a)redhat.com> Gerrit-Reviewer: Kiril Nesenko <knesenko(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>

1 0

Change in ovirt-iso-uploader[ovirt-iso-uploader-3.2]: engine-iso-uploader - notify the user for started upload pro...
by knesenko＠redhat.com 31 Mar '13

31 Mar '13

Kiril Nesenko has submitted this change and it was merged. Change subject: engine-iso-uploader - notify the user for started upload process ...................................................................... engine-iso-uploader - notify the user for started upload process In upload_to_storage_domain notify the user that the upload process has started. Log also the starting of every upload, not only the completion. In upload_to_storage_domain fixed redefinition of built-in 'file' Change-Id: Id45632a662b12d8681dc99b2924105601a0e0d36 Bug-Url: https://bugzilla.redhat.com/785670 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M src/__main__.py 1 file changed, 22 insertions(+), 20 deletions(-) Approvals: Kiril Nesenko: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/13457 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id45632a662b12d8681dc99b2924105601a0e0d36 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-iso-uploader Gerrit-Branch: ovirt-iso-uploader-3.2 Gerrit-Owner: Kiril Nesenko <knesenko(a)redhat.com> Gerrit-Reviewer: Kiril Nesenko <knesenko(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>

1 0

Change in ovirt-iso-uploader[ovirt-iso-uploader-3.2]: engine-iso-uploader - small usage's output change
by knesenko＠redhat.com 31 Mar '13

31 Mar '13

Kiril Nesenko has submitted this change and it was merged. Change subject: engine-iso-uploader - small usage's output change ...................................................................... engine-iso-uploader - small usage's output change The usage printed to the screen now says that upload option requires FILE as mandatory parameter. Change-Id: Ibf303445c4035589322d2b69dbed8d00d11a64f8 Bug-Url: https://bugzilla.redhat.com/785670 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M src/__main__.py 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Kiril Nesenko: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/13456 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ibf303445c4035589322d2b69dbed8d00d11a64f8 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-iso-uploader Gerrit-Branch: ovirt-iso-uploader-3.2 Gerrit-Owner: Kiril Nesenko <knesenko(a)redhat.com> Gerrit-Reviewer: Kiril Nesenko <knesenko(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>

1 0

Change in ovirt-iso-uploader[ovirt-iso-uploader-3.2]: packaging: post ovirt-iso-uploader-3.2.0
by knesenko＠redhat.com 31 Mar '13

31 Mar '13

Kiril Nesenko has submitted this change and it was merged. Change subject: packaging: post ovirt-iso-uploader-3.2.0 ...................................................................... packaging: post ovirt-iso-uploader-3.2.0 Change-Id: I268cfd95cb38c3740d3ab6181a0d28930ab761e7 Signed-off-by: Kiril Nesenko <knesenko(a)redhat.com> --- M configure.ac 1 file changed, 3 insertions(+), 3 deletions(-) Approvals: Alon Bar-Lev: Looks good to me, but someone else must approve Kiril Nesenko: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/13455 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I268cfd95cb38c3740d3ab6181a0d28930ab761e7 Gerrit-PatchSet: 5 Gerrit-Project: ovirt-iso-uploader Gerrit-Branch: ovirt-iso-uploader-3.2 Gerrit-Owner: Kiril Nesenko <knesenko(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Kiril Nesenko <knesenko(a)redhat.com>

1 0

Change in ovirt-image-uploader[ovirt-image-uploader-3.2]: packaging: post ovirt-image-uploader-3.2.1
by knesenko＠redhat.com 31 Mar '13

31 Mar '13

Kiril Nesenko has submitted this change and it was merged. Change subject: packaging: post ovirt-image-uploader-3.2.1 ...................................................................... packaging: post ovirt-image-uploader-3.2.1 Change-Id: I0725c99581caea2230e02a789d8479f862ef8d7f Signed-off-by: Kiril Nesenko <knesenko(a)redhat.com> --- M configure.ac 1 file changed, 3 insertions(+), 3 deletions(-) Approvals: Alon Bar-Lev: Looks good to me, but someone else must approve Kiril Nesenko: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/13453 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I0725c99581caea2230e02a789d8479f862ef8d7f Gerrit-PatchSet: 1 Gerrit-Project: ovirt-image-uploader Gerrit-Branch: ovirt-image-uploader-3.2 Gerrit-Owner: Kiril Nesenko <knesenko(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Kiril Nesenko <knesenko(a)redhat.com>

1 0

Change in ovirt-image-uploader[ovirt-image-uploader-3.2]: packaging: ovirt-image-uploader-3.2.1
by knesenko＠redhat.com 31 Mar '13

31 Mar '13

Kiril Nesenko has submitted this change and it was merged. Change subject: packaging: ovirt-image-uploader-3.2.1 ...................................................................... packaging: ovirt-image-uploader-3.2.1 Change-Id: I51da1fb87afa5f54fa5f426375d94d085951f491 Signed-off-by: Kiril Nesenko <knesenko(a)redhat.com> --- M configure.ac M ovirt-image-uploader.spec.in 2 files changed, 3 insertions(+), 1 deletion(-) Approvals: Alon Bar-Lev: Looks good to me, but someone else must approve Kiril Nesenko: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/13452 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I51da1fb87afa5f54fa5f426375d94d085951f491 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-image-uploader Gerrit-Branch: ovirt-image-uploader-3.2 Gerrit-Owner: Kiril Nesenko <knesenko(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Kiril Nesenko <knesenko(a)redhat.com>

1 0

Change in ovirt-image-uploader[ovirt-image-uploader-3.2]: build: use embedded gettext
by knesenko＠redhat.com 31 Mar '13

31 Mar '13

Kiril Nesenko has submitted this change and it was merged. Change subject: build: use embedded gettext ...................................................................... build: use embedded gettext there are too many variants of version within different distributions for us to support it externally Change-Id: I272e4f0fe58cadf8742ac8a87d77bc560b5d12f4 Signed-off-by: Alon Bar-Lev <alon.barlev(a)gmail.com> --- M .gitignore M Makefile.am M configure.ac M ovirt-image-uploader.spec.in 4 files changed, 5 insertions(+), 3 deletions(-) Approvals: Kiril Nesenko: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/13449 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I272e4f0fe58cadf8742ac8a87d77bc560b5d12f4 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-image-uploader Gerrit-Branch: ovirt-image-uploader-3.2 Gerrit-Owner: Kiril Nesenko <knesenko(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Kiril Nesenko <knesenko(a)redhat.com>

1 0

Change in ovirt-engine[master]: core: Removing CreateComputerAccount command
by yzaslavs＠redhat.com 31 Mar '13

31 Mar '13

Yair Zaslavsky has submitted this change and it was merged. Change subject: core: Removing CreateComputerAccount command ...................................................................... core: Removing CreateComputerAccount command Change-Id: I6224722615cf61b23a35f3eda6db802e5dc1a6bd Signed-off-by: Yair Zaslavsky <yzaslavs(a)redhat.com> --- D backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/CreateComputerAccountCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/adbroker/AdActionType.java D backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/adbroker/LdapCreateComputerAccountCommand.java D backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/adbroker/LdapCreateComputerAccountParameters.java D backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/action/CreateComputerAccountParameters.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/action/VdcActionType.java 6 files changed, 0 insertions(+), 206 deletions(-) Approvals: Yair Zaslavsky: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/13291 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6224722615cf61b23a35f3eda6db802e5dc1a6bd Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com>

1 0

Change in ovirt-image-uploader[ovirt-image-uploader-3.2]: Added example to man page for -N option.
by knesenko＠redhat.com 31 Mar '13

31 Mar '13

Kiril Nesenko has submitted this change and it was merged. Change subject: Added example to man page for -N option. ...................................................................... Added example to man page for -N option. Change-Id: I02d0a4defb248decf8464256befd94fcd3151e48 Bug-Url: https://bugzilla.redhat.com/887566 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M src/engine-image-uploader.8 1 file changed, 8 insertions(+), 0 deletions(-) Approvals: Kiril Nesenko: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/13448 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I02d0a4defb248decf8464256befd94fcd3151e48 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-image-uploader Gerrit-Branch: ovirt-image-uploader-3.2 Gerrit-Owner: Kiril Nesenko <knesenko(a)redhat.com> Gerrit-Reviewer: Kiril Nesenko <knesenko(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>

1 0

Change in ovirt-engine[master]: core: Remove IsVdsWithSameNameExist
by amureini＠redhat.com 31 Mar '13

31 Mar '13

Allon Mureinik has submitted this change and it was merged. Change subject: core: Remove IsVdsWithSameNameExist ...................................................................... core: Remove IsVdsWithSameNameExist IsVdsWithSameNameExistQuery is not used anywhere in the code, and hence should be removed. This patch removes the following: 1. The IsVdsWithSameNameExistQuery class 2. The IsVdsWithSameNameExist constant representing it in VdcQueryType 3. The IsVdsWithSameNameExistParameters class which was only used by the removed query class. 3. The getByVdsName(String) method in VdsStaticDAO (which was only used by the query), its implementation, and its test. 4. The GetVdsStaticByVdsNam stored procedure that was only used by the said DAO function. Change-Id: I676cca49a2cc62c7b0716589ee26d2bcffd3cbb7 Signed-off-by: Allon Mureinik <amureini(a)redhat.com> --- M backend/manager/dbscripts/vds_sp.sql D backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/IsVdsWithSameNameExistQuery.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VdsHandler.java D backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/queries/IsVdsWithSameNameExistParameters.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/VdsStaticDAO.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/VdsStaticDAODbFacadeImpl.java M backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/VdsStaticDAOTest.java 7 files changed, 0 insertions(+), 98 deletions(-) Approvals: Allon Mureinik: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/13290 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I676cca49a2cc62c7b0716589ee26d2bcffd3cbb7 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Alissa Bonas <abonas(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Liron Ar <laravot(a)redhat.com> Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Vered Volansky <vvolansk(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com>

1 0