Change in vdsm[master]: register: Replace log location

[Douglas Schilling Landgraf]

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 
---
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")

[e-mail] 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]

Douglas Schilling Landgraf has posted comments on this change.

Change subject: register: Replace log location
......................................................................

Patch Set 1: Verified+1

[Douglas Schilling Landgraf]

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]

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/

[e-mail] has posted comments on this change.

Change subject: register: Replace log location
......................................................................

Patch Set 1:

* Update tracker::IGNORE, no Bug-Url found