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
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
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
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
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..
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
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
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
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
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.
10 matches
Mail list logo