Bug 950712 - log file overwritten each upload attempt
Summary: log file overwritten each upload attempt
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Red Hat Enterprise Virtualization Manager
Classification: Red Hat
Component: ovirt-image-uploader
Version: 3.1.0
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
: 3.3.0
Assignee: Sandro Bonazzola
QA Contact: Pavel Stehlik
URL:
Whiteboard: integration
Depends On:
Blocks: 1016651
TreeView+ depends on / blocked
 
Reported: 2013-04-10 17:46 UTC by Dave Johnson
Modified: 2015-09-22 13:09 UTC (History)
7 users (show)

Fixed In Version: rhevm-image-uploader-3.3.0-0.1.rc.el6ev
Doc Type: Bug Fix
Doc Text:
Every time an image was uploaded, the existing log file was overwritten, making it difficult to debug issues. Now, messages are logged to timestamped files, to avoid overwriting existing ones. A log rotation was also introduced to compress old logs.
Clone Of:
: 1016651 (view as bug list)
Environment:
Last Closed: 2014-01-21 16:32:21 UTC
oVirt Team: ---
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Red Hat Product Errata RHBA-2014:0078 0 normal SHIPPED_LIVE rhevm-image-uploader bug fix and enhancement update 2014-01-21 20:36:35 UTC
oVirt gerrit 18290 0 None None None Never
oVirt gerrit 18321 0 None None None Never

Description Dave Johnson 2013-04-10 17:46:27 UTC
Description of problem:
========================================================
Each time you attempt to upload a image, the existing log file gets overwritten which makes debugging issues harder when you execute multiple attempts to upload.

Either append or save existing off to a timestamp prefixed file would be a little more customer friendly.


Version-Release number of selected component (if applicable):
====================================================================
rhevm-image-uploader-3.1.0-7.el6ev.noarch


How reproducible:   
================
100%

Steps to Reproduce:
==================================
1.  run several uploads in succession and see the log file only tracks the last attempt

Comment 1 Sandro Bonazzola 2013-08-19 14:14:31 UTC
Patch pushed on master branch.
Note that also log-collector and iso-uploader are affected by the same bug.
I'll open one bug for each of them.

Comment 2 Sandro Bonazzola 2013-08-20 08:10:43 UTC
Log messages to timestamped files as default, avoiding to overwrite existing ones.

The default location of the logs was changed, to improve /var/log/ovirt-engine sanity.

/var/log/ovirt-engine -> /var/log/ovirt-engine/ovirt-image-uploader

A log rotation was introduced in order to compress old logs.

Comment 3 Sandro Bonazzola 2013-08-20 11:21:25 UTC
merged upstream master, pushed to 3.3 branch

Comment 4 Sandro Bonazzola 2013-08-21 05:49:38 UTC
merged upstream 3.3 branch

Comment 6 David Botzer 2013-08-27 09:06:59 UTC
3.3/is11
I get a new file for each image-uploader command is this correct ?
=======================
>ovirt-image-uploader-20130827113507.log
2013-08-27 11:35:18::ERROR::rhevm-image-uploader::1335::root:: not a gzip file

>ovirt-image-uploader-20130827120229.log
2013-08-27 12:02:40::ERROR::rhevm-image-uploader::1335::root:: empty header

-rw-r--r--. 1 root root  192 Aug 27 11:09 ovirt-image-uploader-20130827110954.log
-rw-r--r--. 1 root root  192 Aug 27 11:11 ovirt-image-uploader-20130827111114.log
-rw-r--r--. 1 root root  192 Aug 27 11:11 ovirt-image-uploader-20130827111131.log
-rw-r--r--. 1 root root  192 Aug 27 11:11 ovirt-image-uploader-20130827111133.log
-rw-r--r--. 1 root root  192 Aug 27 11:11 ovirt-image-uploader-20130827111135.log
-rw-r--r--. 1 root root  192 Aug 27 11:11 ovirt-image-uploader-20130827111136.log
-rw-r--r--. 1 root root   99 Aug 27 11:12 ovirt-image-uploader-20130827111242.log
-rw-r--r--. 1 root root    0 Aug 27 11:16 ovirt-image-uploader-20130827111602.log
-rw-r--r--. 1 root root  209 Aug 27 11:17 ovirt-image-uploader-20130827111750.log
-rw-r--r--. 1 root root    0 Aug 27 11:17 ovirt-image-uploader-20130827111757.log
-rw-r--r--. 1 root root  110 Aug 27 11:18 ovirt-image-uploader-20130827111845.log
-rw-r--r--. 1 root root   79 Aug 27 11:35 ovirt-image-uploader-20130827113507.log
-rw-r--r--. 1 root root   76 Aug 27 12:02 ovirt-image-uploader-20130827120229.log
-rw-r--r--. 1 root root  109 Aug 27 12:04 ovirt-image-uploader-20130827120434.log
-rw-r--r--. 1 root root   76 Aug 27 12:05 ovirt-image-uploader-20130827120456.log
-rw-r--r--. 1 root root   76 Aug 27 12:05 ovirt-image-uploader-20130827120528.log

Comment 7 Sandro Bonazzola 2013-09-04 08:49:07 UTC
(In reply to David Botzer from comment #6)
> 3.3/is11
> I get a new file for each image-uploader command is this correct ?

Yes a new log file is created for each image-uploader execution. It is correct.

Comment 8 David Botzer 2013-09-04 09:03:35 UTC
Fixed, 3.3/is11

Comment 11 errata-xmlrpc 2014-01-21 16:32:21 UTC
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

http://rhn.redhat.com/errata/RHBA-2014-0078.html


Note You need to log in before you can comment on or make changes to this bug.