September 2013 - Engine-commits - oVirt List Archives

Change in ovirt-iso-uploader[master]: upload: nfs: handle rename errors
by sbonazzo＠redhat.com 12 Sep '13

12 Sep '13

Sandro Bonazzola has submitted this change and it was merged. Change subject: upload: nfs: handle rename errors ...................................................................... upload: nfs: handle rename errors Uploading to a NFS destination, if an error were encountered while renaming the used temporary file to the final destination, the application displayed: uploaded successfully. Now the application exits with exit status 3 (upload error) and display an error message. Added also a debug log for recording the rename action in the verbose log. Change-Id: I8fc993d0fdc36924cbf3dd7ed50569ccdcfc75b5 Bug-Url: https://bugzilla.redhat.com/1005100 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M src/__main__.py 1 file changed, 42 insertions(+), 12 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/19095 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8fc993d0fdc36924cbf3dd7ed50569ccdcfc75b5 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-iso-uploader Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Alex Lourie <alourie(a)redhat.com> Gerrit-Reviewer: Keith Robertson <kroberts(a)redhat.com> Gerrit-Reviewer: Kiril Nesenko <knesenko(a)redhat.com> Gerrit-Reviewer: Ofer Schreiber <oschreib(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>

1 0

Change in ovirt-engine[ovirt-engine-3.3.0]: pacakging: bump rpm release to 3
by oschreib＠redhat.com 12 Sep '13

12 Sep '13

Ofer Schreiber has submitted this change and it was merged. Change subject: pacakging: bump rpm release to 3 ...................................................................... pacakging: bump rpm release to 3 Bump release in order to force upgrade of current public RPM Change-Id: Id2bb980b16231d5d33f81c81c00ddc82cfe68ee7 Signed-off-by: Ofer Schreiber <oschreib(a)redhat.com> --- M version.mak 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Alon Bar-Lev: Looks good to me, but someone else must approve Ofer Schreiber: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/19181 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id2bb980b16231d5d33f81c81c00ddc82cfe68ee7 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.3.0 Gerrit-Owner: Ofer Schreiber <oschreib(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Ofer Schreiber <oschreib(a)redhat.com>

1 0

Change in ovirt-engine[master]: engine: virtio -> VirtIO
by tjelinek＠redhat.com 12 Sep '13

12 Sep '13

Tomas Jelinek has submitted this change and it was merged. Change subject: engine: virtio -> VirtIO ...................................................................... engine: virtio -> VirtIO This patch correct wrong VirtIO case. Change-Id: Ifc2b02e02f1656b63dba84ffbe998f4a1db8a0a3 Bug-Url: https://bugzilla.redhat.com/999977 Signed-off-by: Frantisek Kobzik <fkobzik(a)redhat.com> --- M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/utils/FeaturesHelper.java M frontend/webadmin/modules/frontend/src/main/java/org/ovirt/engine/ui/frontend/AppErrors.java M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/CommonApplicationConstants.java 3 files changed, 4 insertions(+), 4 deletions(-) Approvals: Tomas Jelinek: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/18658 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ifc2b02e02f1656b63dba84ffbe998f4a1db8a0a3 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Frank Kobzik <fkobzik(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Frank Kobzik <fkobzik(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: engine: VM NewPool-1 was restarted on Host <UNKNOWN>
by tjelinek＠redhat.com 12 Sep '13

12 Sep '13

Tomas Jelinek has submitted this change and it was merged. Change subject: engine: VM NewPool-1 was restarted on Host <UNKNOWN> ...................................................................... engine: VM NewPool-1 was restarted on Host <UNKNOWN> If the host has been started as prestarted the following event log has been logged: VM NewPool-1 was restarted on Host <UNKNOWN> The problem was that the Vm has been started as stateless. This first triggers the statelessVmTreatment() which does not schedule the host but will already be logged (e.g. the host will be unknown). Fixed by checking if the host has already been scheduled and if not than a different message will be logged (a similar to the one when the VM is started as stateless by the user). Change-Id: I60a74762984dc82154d0f8e073c60f631da229f0 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=966619 Signed-off-by: Tomas Jelinek <tjelinek(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RunVmCommand.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/AuditLogType.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dal/dbbroker/auditloghandling/AuditLogDirector.java M backend/manager/modules/dal/src/main/resources/bundles/AuditLogMessages.properties 4 files changed, 17 insertions(+), 4 deletions(-) Approvals: Tomas Jelinek: Verified Omer Frenkel: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/18726 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I60a74762984dc82154d0f8e073c60f631da229f0 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[ovirt-engine-3.3]: packaging: engine-backup - support old md5sum
by oschreib＠redhat.com 12 Sep '13

12 Sep '13

Ofer Schreiber has submitted this change and it was merged. Change subject: packaging: engine-backup - support old md5sum ...................................................................... packaging: engine-backup - support old md5sum Removed '--strict' md5sum option, which isn't supported on earlier versions of md5sum. Change-Id: I723bc81e84be05aa2fb3ca2ceb8210a2bfe7adc9 Signed-off-by: Ofer Schreiber <oschreib(a)redhat.com> Bug-URL: https://bugzilla.redhat.com/1003869 --- M packaging/bin/engine-backup.sh 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Ofer Schreiber: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/19127 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I723bc81e84be05aa2fb3ca2ceb8210a2bfe7adc9 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.3 Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Ofer Schreiber <oschreib(a)redhat.com>

1 0

Change in ovirt-engine[ovirt-engine-3.3]: build: commons-lang is available at dependencies module
by oschreib＠redhat.com 12 Sep '13

12 Sep '13

Ofer Schreiber has submitted this change and it was merged. Change subject: build: commons-lang is available at dependencies module ...................................................................... build: commons-lang is available at dependencies module Change-Id: Id8b531d2cf770d1efb130852373bb20a254e02a0 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M backend/manager/modules/root/pom.xml 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: Ofer Schreiber: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/19125 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id8b531d2cf770d1efb130852373bb20a254e02a0 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.3 Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Ofer Schreiber <oschreib(a)redhat.com>

1 0

Change in ovirt-engine[master]: core: USER_FINISHED[_FAILED]_REMOVE_DISK Grammar
by amureini＠redhat.com 12 Sep '13

12 Sep '13

Allon Mureinik has submitted this change and it was merged. Change subject: core: USER_FINISHED[_FAILED]_REMOVE_DISK Grammar ...................................................................... core: USER_FINISHED[_FAILED]_REMOVE_DISK Grammar Fixed messages text. Change-Id: Ie898ba54ab5b673eefb594350df5ee22cf145de0 Signed-off-by: Allon Mureinik <amureini(a)redhat.com> --- M backend/manager/modules/dal/src/main/resources/bundles/AuditLogMessages.properties 1 file changed, 2 insertions(+), 2 deletions(-) Approvals: Allon Mureinik: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/19179 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie898ba54ab5b673eefb594350df5ee22cf145de0 Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Allon Mureinik <amureini(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: Tal Nisan <tnisan(a)redhat.com>

1 0

Change in ovirt-engine[master]: core: setting singleQxlPci to false when importing external VMs
by oourfali＠redhat.com 12 Sep '13

12 Sep '13

Oved Ourfali has submitted this change and it was merged. Change subject: core: setting singleQxlPci to false when importing external VMs ...................................................................... core: setting singleQxlPci to false when importing external VMs Change-Id: I8ef10def74461d4fdd0e4c596eea1a157e6463e8 Bug-Url: https://bugzilla.redhat.com/978268 Signed-off-by: Oved Ourfali <oourfali(a)redhat.com> --- M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VdsUpdateRunTimeInfo.java 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: Shahar Havivi: Looks good to me, approved Oved Ourfali: Verified -- To view, visit http://gerrit.ovirt.org/19180 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8ef10def74461d4fdd0e4c596eea1a157e6463e8 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Shahar Havivi <shavivi(a)redhat.com>

1 0

Change in ovirt-engine[master]: restapi: Add subcollections for network resource
by masayag＠redhat.com 12 Sep '13

12 Sep '13

Moti Asayag has submitted this change and it was merged. Change subject: restapi: Add subcollections for network resource ...................................................................... restapi: Add subcollections for network resource The patch adds the vnicprofiles and permissions subcollections for a single network resource. Change-Id: I8b8d8b144dcac69d5355bd26f8ffacb9e0fdfc62 Bug-Url: https://bugzilla.redhat.com/1003881 Signed-off-by: Moti Asayag <masayag(a)redhat.com> --- M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/AbstractBackendNetworkResource.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendNetworkResource.java 2 files changed, 3 insertions(+), 3 deletions(-) Approvals: Michael Pasternak: Looks good to me, approved Moti Asayag: Verified -- To view, visit http://gerrit.ovirt.org/18839 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8b8d8b144dcac69d5355bd26f8ffacb9e0fdfc62 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: engine: Add error message for ifup failure
by masayag＠redhat.com 12 Sep '13

12 Sep '13

Moti Asayag has submitted this change and it was merged. Change subject: engine: Add error message for ifup failure ...................................................................... engine: Add error message for ifup failure The patch adds a missing error code from vdsm for a failure to bring an interface up. Change-Id: I07adde3368c31319bed48839d089ba3ac48cd424 Bug-Url: https://bugzilla.redhat.com/990938 Signed-off-by: Moti Asayag <masayag(a)redhat.com> --- M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/errors/VdcBllErrors.java M backend/manager/modules/dal/src/main/resources/bundles/VdsmErrors.properties M frontend/webadmin/modules/frontend/src/main/java/org/ovirt/engine/ui/frontend/VdsmErrors.java M frontend/webadmin/modules/userportal-gwtp/src/main/resources/org/ovirt/engine/ui/frontend/VdsmErrors.properties M frontend/webadmin/modules/webadmin/src/main/resources/org/ovirt/engine/ui/frontend/VdsmErrors.properties 5 files changed, 6 insertions(+), 0 deletions(-) Approvals: Moti Asayag: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/19111 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I07adde3368c31319bed48839d089ba3ac48cd424 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Mike Kolesnik <mkolesni(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0