Zhou Zheng Sheng has posted comments on this change.

Change subject: Seperating supervdsm log to supervdsm.log file
......................................................................


Patch Set 2: (1 inline comment)

....................................................
File vdsm/supervdsmServer.py
Line 356:     try:
Line 357:         f = 
"%(threadName)s::%(levelname)s::%(asctime)s::%(module)s::" \
Line 358:             "%(lineno)d::%(name)s::(%(funcName)s) %(message)s"
Line 359:         logfile = P_VDSM_LOG + "/suprevdsm.log"
Line 360:         logging.basicConfig(filename=logfile,
Yes, we just need a new handler section in logger.conf to deliver the log to 
svdsm.log, and assign this handler to SuperVdsm logger section.
Line 361:                             level=logging.DEBUG,
Line 362:                             format=f,)
Line 363:     except:
Line 364:         logging.basicConfig(filename='/dev/stdout', filemode='w+',


--
To view, visit http://gerrit.ovirt.org/12940
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I2e5fb4a2cd5421671f942120b6718c1bb20b9e60
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Yaniv Bronhaim <ybron...@redhat.com>
Gerrit-Reviewer: Adam Litke <a...@us.ibm.com>
Gerrit-Reviewer: Alon Bar-Lev <alo...@redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com>
Gerrit-Reviewer: Saggi Mizrahi <smizr...@redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybron...@redhat.com>
Gerrit-Reviewer: Zhou Zheng Sheng <zhshz...@linux.vnet.ibm.com>
Gerrit-Reviewer: oVirt Jenkins CI Server
_______________________________________________
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to