Change in vdsm[master]: vdscli: do not use the warnings module

2016-09-19 Thread fromani
Francesco Romani has abandoned this change. Change subject: vdscli: do not use the warnings module .. Abandoned replaced by https://gerrit.ovirt.org/#/c/64100/2 -- To view, visit https://gerrit.ovirt.org/64117 To

Change in vdsm[master]: vdscli: do not use the warnings module

2016-09-19 Thread automation
gerrit-hooks has posted comments on this change. Change subject: vdscli: do not use the warnings module .. Patch Set 3: * Update tracker: IGNORE, no Bug-Url found -- To view, visit https://gerrit.ovirt.org/64117 To

Change in vdsm[master]: vdscli: do not use the warnings module

2016-09-19 Thread automation
gerrit-hooks has posted comments on this change. Change subject: vdscli: do not use the warnings module .. Patch Set 3: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header

Change in vdsm[master]: vdscli: do not use the warnings module

2016-09-19 Thread automation
gerrit-hooks has posted comments on this change. Change subject: vdscli: do not use the warnings module .. Patch Set 2: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header

Change in vdsm[master]: vdscli: do not use the warnings module

2016-09-19 Thread msivak
Martin Sivák has posted comments on this change. Change subject: vdscli: do not use the warnings module .. Patch Set 1: Ah, Nir is right, because we log during import. And the logging subsystem won't be ready at that time..

Change in vdsm[master]: vdscli: do not use the warnings module

2016-09-19 Thread msivak
Martin Sivák has posted comments on this change. Change subject: vdscli: do not use the warnings module .. Patch Set 1: Nir: the logger will be unconfigured, but messages should be propagated by default and end up in the root

Change in vdsm[master]: vdscli: do not use the warnings module

2016-09-19 Thread msivak
Martin Sivák has posted comments on this change. Change subject: vdscli: do not use the warnings module .. Patch Set 1: (2 comments) https://gerrit.ovirt.org/#/c/64117/1/lib/vdsm/vdscli.py File lib/vdsm/vdscli.py: PS1, Line

Change in vdsm[master]: vdscli: do not use the warnings module

2016-09-19 Thread nsoffer
Nir Soffer has posted comments on this change. Change subject: vdscli: do not use the warnings module .. Patch Set 1: Code-Review-1 Logging to secret logger in a library means the log would show in the program importing this

Change in vdsm[master]: vdscli: do not use the warnings module

2016-09-19 Thread automation
gerrit-hooks has posted comments on this change. Change subject: vdscli: do not use the warnings module .. Patch Set 1: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header

Change in vdsm[master]: vdscli: do not use the warnings module

2016-09-19 Thread fromani
Francesco Romani has uploaded a new change for review. Change subject: vdscli: do not use the warnings module .. vdscli: do not use the warnings module Due the possible memory leaks, switch to our new 'devel' logger.