Change in vdsm[master]: api: emit warnings only if configured

2016-08-10 Thread fromani
Francesco Romani has uploaded a new change for review. Change subject: api: emit warnings only if configured .. api: emit warnings only if configured The python standard module `warnings` implement filtering for warnings, to all

Change in vdsm[master]: api: emit warnings only if configured

2016-08-10 Thread automation
gerrit-hooks has posted comments on this change. Change subject: api: emit warnings only if configured .. Patch Set 1: * #1364149::Update tracker: OK * Check Bug-Url::OK * Check Public Bug::#1364149::OK, public bug * Check Prod

Change in vdsm[master]: api: emit warnings only if configured

2016-08-10 Thread michal . skrivanek
Michal Skrivanek has posted comments on this change. Change subject: api: emit warnings only if configured .. Patch Set 1: Code-Review+1 -- To view, visit https://gerrit.ovirt.org/62186 To unsubscribe, visit https://gerrit.ovi

Change in vdsm[master]: api: emit warnings only if configured

2016-08-10 Thread piotr . kliczewski
Piotr Kliczewski has posted comments on this change. Change subject: api: emit warnings only if configured .. Patch Set 1: Code-Review+2 With this patch validation of request/responses goes down the drain. We could not provide

Change in vdsm[master]: api: emit warnings only if configured

2016-08-10 Thread michal . skrivanek
Michal Skrivanek has posted comments on this change. Change subject: api: emit warnings only if configured .. Patch Set 1: yeah:) though I suppose we will soon replace the logging with something better globally, and re-enable

Change in vdsm[master]: api: emit warnings only if configured

2016-08-10 Thread fromani
Francesco Romani has posted comments on this change. Change subject: api: emit warnings only if configured .. Patch Set 1: Piotr, I don't want to cripple validation of JSONRPC unless I really need to do so. As part of verifica

Change in vdsm[master]: api: emit warnings only if configured

2016-08-10 Thread fromani
Francesco Romani has posted comments on this change. Change subject: api: emit warnings only if configured .. Patch Set 1: Verified+1 I verified this patch on a QE box which presents the leakage, on top of 4.18.6. So this appl

Change in vdsm[master]: api: emit warnings only if configured

2016-08-10 Thread nsoffer
Nir Soffer has submitted this change and it was merged. Change subject: api: emit warnings only if configured .. api: emit warnings only if configured The python standard module `warnings` implement filtering for warnings, to a

Change in vdsm[master]: api: emit warnings only if configured

2016-08-10 Thread automation
gerrit-hooks has posted comments on this change. Change subject: api: emit warnings only if configured .. Patch Set 2: * #1364149::Update tracker: OK * Set MODIFIED::bug 1364149#1364149OK -- To view, visit https://ger