Douglas Schilling Landgraf has uploaded a new change for review.
Change subject: register: Replace log location ......................................................................
register: Replace log location
The register tool is integrated to vdsm-tool but can be used without vdsm. This patch replaces the log location.
Change-Id: I884c7c09dc16829752dc3801d1977e0097733939 Signed-off-by: Douglas Schilling Landgraf dougsland@redhat.com --- M lib/vdsm/tool/register.py 1 file changed, 2 insertions(+), 2 deletions(-)
git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/31/42831/1
diff --git a/lib/vdsm/tool/register.py b/lib/vdsm/tool/register.py index 3ac3fc6..9b59e5f 100644 --- a/lib/vdsm/tool/register.py +++ b/lib/vdsm/tool/register.py @@ -179,13 +179,13 @@ def _set_logger(self): """ The logging settings - Saving log in: /var/log/vdsm/register.log + Saving log in: /var/log/ovirt-register.log """ logger = logging.getLogger(__name__) logger.setLevel(logging.DEBUG) logger.propagate = False
- fh = logging.FileHandler("/var/log/vdsm/register.log") + fh = logging.FileHandler("/var/log/ovirt-register.log") fh.setLevel(logging.DEBUG) debug_fmt = logging.Formatter("%(asctime)s %(message)s", "%m/%d/%Y %I:%M:%S %p")
automation@ovirt.org has posted comments on this change.
Change subject: register: Replace log location ......................................................................
Patch Set 1:
* Update tracker::IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' and is a valid url. * Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.5', 'ovirt-3.4', 'ovirt-3.3'])
Douglas Schilling Landgraf has posted comments on this change.
Change subject: register: Replace log location ......................................................................
Patch Set 1: Verified+1
Douglas Schilling Landgraf has posted comments on this change.
Change subject: register: Replace log location ......................................................................
Patch Set 1:
Dan, what do you say?
Douglas Schilling Landgraf has abandoned this change.
Change subject: register: Replace log location ......................................................................
Abandoned
Hi Dan,
I will keep the log in the current location as this verb is integrated to vdsm. We have generic ovirt-register project which has option for specify log location and register as non root.
https://admin.fedoraproject.org/pkgdb/package/python-ovirt-register/
automation@ovirt.org has posted comments on this change.
Change subject: register: Replace log location ......................................................................
Patch Set 1:
* Update tracker::IGNORE, no Bug-Url found
vdsm-patches@lists.fedorahosted.org