Change in vdsm[master]: vdsm: Rely on system for logrotation

2016-09-19 Thread nsoffer
Nir Soffer has posted comments on this change. Change subject: vdsm: Rely on system for logrotation .. Patch Set 3: Please send another patch documenting why we are not using logrotate in the standard way, with url to the

Change in vdsm[master]: vdsm: Rely on system for logrotation

2016-09-19 Thread Tomas Golembiovsky
Tomas Golembiovsky has posted comments on this change. Change subject: vdsm: Rely on system for logrotation .. Patch Set 3: Code-Review-1 Thanks Dan. Somehow I have missed this change when browsing through related history.

Change in vdsm[master]: vdsm: Rely on system for logrotation

2016-09-19 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: vdsm: Rely on system for logrotation .. Patch Set 3: I am not 100% that this still applies, but please discuss

Change in vdsm[master]: vdsm: Rely on system for logrotation

2016-09-18 Thread nsoffer
Nir Soffer has posted comments on this change. Change subject: vdsm: Rely on system for logrotation .. Patch Set 3: Code-Review+1 (1 comment) https://gerrit.ovirt.org/#/c/63682/3/vdsm/vdsm-remove-logs File

Change in vdsm[master]: vdsm: Rely on system for logrotation

2016-09-15 Thread automation
gerrit-hooks has posted comments on this change. Change subject: vdsm: Rely on system for logrotation .. Patch Set 3: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header matches

Change in vdsm[master]: vdsm: Rely on system for logrotation

2016-09-15 Thread automation
gerrit-hooks has posted comments on this change. Change subject: vdsm: Rely on system for logrotation .. Patch Set 2: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header matches

Change in vdsm[master]: vdsm: Rely on system for logrotation

2016-09-14 Thread Tomas Golembiovsky
Tomas Golembiovsky has posted comments on this change. Change subject: vdsm: Rely on system for logrotation .. Patch Set 1: (4 comments) https://gerrit.ovirt.org/#/c/63682/1//COMMIT_MSG Commit Message: Line 10: is no real

Change in vdsm[master]: vdsm: Rely on system for logrotation

2016-09-13 Thread nsoffer
Nir Soffer has posted comments on this change. Change subject: vdsm: Rely on system for logrotation .. Patch Set 1: (4 comments) https://gerrit.ovirt.org/#/c/63682/1//COMMIT_MSG Commit Message: Line 10: is no real reason to

Change in vdsm[master]: vdsm: Rely on system for logrotation

2016-09-13 Thread Tomas Golembiovsky
Tomas Golembiovsky has posted comments on this change. Change subject: vdsm: Rely on system for logrotation .. Patch Set 1: (4 comments) https://gerrit.ovirt.org/#/c/63682/1//COMMIT_MSG Commit Message: Line 5: CommitDate:

Change in vdsm[master]: vdsm: Rely on system for logrotation

2016-09-13 Thread piotr . kliczewski
Piotr Kliczewski has posted comments on this change. Change subject: vdsm: Rely on system for logrotation .. Patch Set 1: (2 comments) https://gerrit.ovirt.org/#/c/63682/1//COMMIT_MSG Commit Message: Line 5: CommitDate:

Change in vdsm[master]: vdsm: Rely on system for logrotation

2016-09-12 Thread nsoffer
Nir Soffer has posted comments on this change. Change subject: vdsm: Rely on system for logrotation .. Patch Set 1: (4 comments) https://gerrit.ovirt.org/#/c/63682/1//COMMIT_MSG Commit Message: Line 10: is no real reason to

Change in vdsm[master]: vdsm: Rely on system for logrotation

2016-09-12 Thread fromani
Francesco Romani has posted comments on this change. Change subject: vdsm: Rely on system for logrotation .. Patch Set 1: Code-Review+1 initial review -- To view, visit https://gerrit.ovirt.org/63682 To unsubscribe, visit

Change in vdsm[master]: vdsm: Rely on system for logrotation

2016-09-12 Thread automation
gerrit-hooks has posted comments on this change. Change subject: vdsm: Rely on system for logrotation .. Patch Set 1: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header matches

Change in vdsm[master]: vdsm: Rely on system for logrotation

2016-09-12 Thread Tomas Golembiovsky
Tomas Golembiovsky has uploaded a new change for review. Change subject: vdsm: Rely on system for logrotation .. vdsm: Rely on system for logrotation When logrotate is installed it is normaly run periodicaly by cron. There is