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 <dougsland(a)redhat.com&gt;
---
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")


-- 
To view, visit https://gerrit.ovirt.org/42831
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I884c7c09dc16829752dc3801d1977e0097733939
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Douglas Schilling Landgraf <dougsland(a)redhat.com&gt;

automation(a)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'])

-- 
To view, visit https://gerrit.ovirt.org/42831
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I884c7c09dc16829752dc3801d1977e0097733939
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Douglas Schilling Landgraf <dougsland(a)redhat.com&gt;
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No

[Douglas Schilling Landgraf]

Douglas Schilling Landgraf has posted comments on this change.

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


Patch Set 1: Verified+1

-- 
To view, visit https://gerrit.ovirt.org/42831
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I884c7c09dc16829752dc3801d1977e0097733939
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Douglas Schilling Landgraf <dougsland(a)redhat.com&gt;
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com&gt;
Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland(a)redhat.com&gt;
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No

[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?

-- 
To view, visit https://gerrit.ovirt.org/42831
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I884c7c09dc16829752dc3801d1977e0097733939
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Douglas Schilling Landgraf <dougsland(a)redhat.com&gt;
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com&gt;
Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland(a)redhat.com&gt;
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No

[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/

-- 
To view, visit https://gerrit.ovirt.org/42831
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: abandon
Gerrit-Change-Id: I884c7c09dc16829752dc3801d1977e0097733939
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Douglas Schilling Landgraf <dougsland(a)redhat.com&gt;
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com&gt;
Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland(a)redhat.com&gt;
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: automation(a)ovirt.org

automation(a)ovirt.org has posted comments on this change.

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


Patch Set 1:

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

-- 
To view, visit https://gerrit.ovirt.org/42831
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I884c7c09dc16829752dc3801d1977e0097733939
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Douglas Schilling Landgraf <dougsland(a)redhat.com&gt;
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com&gt;
Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland(a)redhat.com&gt;
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No