Change in vdsm[master]: net: api: replace custom logger with logging

danken at redhat.com danken at redhat.com
Thu Mar 10 06:52:43 UTC 2016


Dan Kenigsberg has submitted this change and it was merged.

Change subject: net: api: replace custom logger with logging
......................................................................


net: api: replace custom logger with logging

We use one log file for all supervdsm logs, there is no need to
use special logger. Also custom logger name is not necessary
while we already log modules' names.

Change-Id: Ia05f237dedcf932028169e4d8fd336b42dbe5125
Signed-off-by: Petr Horáček <phoracek at redhat.com>
Reviewed-on: https://gerrit.ovirt.org/54350
Continuous-Integration: Jenkins CI
Reviewed-by: Edward Haas <edwardh at redhat.com>
Reviewed-by: Ondřej Svoboda <osvoboda at redhat.com>
Reviewed-by: Dan Kenigsberg <danken at redhat.com>
---
M lib/vdsm/network/api.py
M lib/vdsm/network/legacy_switch.py
2 files changed, 36 insertions(+), 38 deletions(-)

Approvals:
  Ondřej Svoboda: Looks good to me, but someone else must approve
  Jenkins CI: Passed CI tests
  Petr Horáček: Verified
  Dan Kenigsberg: Looks good to me, approved
  Edward Haas: Looks good to me, but someone else must approve



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ia05f237dedcf932028169e4d8fd336b42dbe5125
Gerrit-PatchSet: 6
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Petr Horáček <phoracek at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Edward Haas <edwardh at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Ondřej Svoboda <osvoboda at redhat.com>
Gerrit-Reviewer: Petr Horáček <phoracek at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>


More information about the vdsm-patches mailing list