Change in ovirt-iso-uploader[master]: upload: nfs: handle rename errors
by sbonazzo@redhat.com
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>
11 years, 3 months
Change in ovirt-engine[ovirt-engine-3.3.0]: pacakging: bump rpm release to 3
by oschreib@redhat.com
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>
11 years, 3 months
Change in ovirt-engine[master]: engine: virtio -> VirtIO
by tjelinek@redhat.com
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
11 years, 3 months
Change in ovirt-engine[master]: engine: VM NewPool-1 was restarted on Host <UNKNOWN>
by tjelinek@redhat.com
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
11 years, 3 months
Change in ovirt-engine[ovirt-engine-3.3]: packaging: engine-backup - support old md5sum
by oschreib@redhat.com
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>
11 years, 3 months
Change in ovirt-engine[ovirt-engine-3.3]: build: commons-lang is available at dependencies module
by oschreib@redhat.com
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>
11 years, 3 months
Change in ovirt-engine[master]: core: USER_FINISHED[_FAILED]_REMOVE_DISK Grammar
by amureini@redhat.com
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>
11 years, 3 months
Change in ovirt-engine[master]: core: setting singleQxlPci to false when importing external VMs
by oourfali@redhat.com
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>
11 years, 3 months
Change in ovirt-engine[master]: restapi: Add subcollections for network resource
by masayag@redhat.com
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
11 years, 3 months
Change in ovirt-engine[master]: engine: Add error message for ifup failure
by masayag@redhat.com
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
11 years, 3 months