Re: [ovirt-devel] [VDSM] Excessive warnings in vdsm log

2016-05-18 Thread Piotr Kliczewski 
On Wed, May 18, 2016 at 1:33 PM, Vinzenz Feenstra 
wrote:

>
> On May 18, 2016, at 1:31 PM, Piotr Kliczewski  wrote:
>
> Please see [1][2]. We are seeing build issues for f23 so the build
> failures are not related.
> I asked Gil from infra to take a look.
>
>
I have update from Gil:

 rpm nightly failed today. The ioproces and python-ioprocess are
corrupted.
 ok
 will see what's the status of the nightly. Sandro was working on
fixing that
 great, thank you!


> +1 :-)
>
>
> Thanks,
> Piotr
>
> [1] https://gerrit.ovirt.org/#/c/57618/
> [2] https://gerrit.ovirt.org/#/c/57623
>
> On Wed, May 18, 2016 at 1:29 PM, Vinzenz Feenstra 
> wrote:
>
>>
>> On May 18, 2016, at 1:23 PM, Dan Kenigsberg  wrote:
>>
>> On Tue, May 17, 2016 at 08:44:11AM +0200, Piotr Kliczewski wrote:
>>
>> Nir,
>>
>> The warnings were added to annoy people so we could keep the schema align
>> with the code.
>> I think that we should use this opportunity to push fixes instead of
>> disabling it.
>>
>> Thanks,
>> Piotr
>>
>> On Mon, May 16, 2016 at 10:09 PM, Nir Soffer  wrote:
>>
>> Hi all,
>>
>> Since data verification patches were merged, vdsm logs is spammed with
>> useless warnings (see bellow).
>>
>> This spam make it harder to debug vdsm.
>>
>> Please add configuration variable to enable this warnings, and make
>> them disabled by default.
>>
>> Thanks,
>> Nir
>>
>> 
>> jsonrpc.Executor/0::WARNING::2016-05-16
>> 23:05:21,719::schemaapi::140::SchemaCache::(_report_inconsistency)
>> Following parameters ['ksmMergeAcrossNodes', 'haStats'] were not
>> recognized
>> jsonrpc.Executor/0::WARNING::2016-05-16
>> 23:05:21,720::schemaapi::140::SchemaCache::(_report_inconsistency)
>> Parameter cpuUserVdsmd is not float type
>> jsonrpc.Executor/0::WARNING::2016-05-16
>> 23:05:21,720::schemaapi::140::SchemaCache::(_report_inconsistency)
>> Parameter rxRate is not float type
>>
>>
>> I'm officially annoyed ;-)
>> we can stop report rxRate and txRate now, as engine-3.6 computes them on
>> its own.
>>
>> Marcin, can you drop them from the code and schema?
>>
>> jsonrpc.Executor/0::WARNING::2016-05-16
>> 23:05:21,720::schemaapi::140::SchemaCache::(_report_inconsistency)
>> Parameter cpuLoad is not float type
>> jsonrpc.Executor/0::WARNING::2016-05-16
>> 23:05:21,720::schemaapi::140::SchemaCache::(_report_inconsistency)
>> Parameter memUsed is not uint type
>> jsonrpc.Executor/0::WARNING::2016-05-16
>> 23:05:21,720::schemaapi::140::SchemaCache::(_report_inconsistency)
>> Parameter cpuIdle is not float type
>> jsonrpc.Executor/0::WARNING::2016-05-16
>> 23:05:21,721::schemaapi::140::SchemaCache::(_report_inconsistency)
>> Parameter txRate is not float type
>> jsonrpc.Executor/0::WARNING::2016-05-16
>> 23:05:21,721::schemaapi::140::SchemaCache::(_report_inconsistency)
>> Parameter txDropped is not uint type
>> jsonrpc.Executor/0::WARNING::2016-05-16
>> 23:05:21,721::schemaapi::140::SchemaCache::(_report_inconsistency)
>> Parameter elapsedTime is not uint type
>> jsonrpc.Executor/0::WARNING::2016-05-16
>> 23:05:21,721::schemaapi::140::SchemaCache::(_report_inconsistency)
>> Parameter netConfigDirty is not boolean type
>> jsonrpc.Executor/0::WARNING::2016-05-16
>> 23:05:21,721::schemaapi::140::SchemaCache::(_report_inconsistency)
>> Parameter rxErrors is not uint type
>>
>>
>> Piotr, how can we specify in the schema the very common type of "string
>> that happens to include only unsigned decimal number”?
>>
>>
>> I remember that we talked about that with Piotr and that the best
>> solution would be a custom type for that, so that it can be checked
>> e.g. struint and strint as types, that means that those are integers
>> (signed/unsigned however passed as string) (Due to historical XMLRPC
>> reasons)
>>
>>
>> ___
>> Devel mailing list
>> Devel@ovirt.org
>> http://lists.ovirt.org/mailman/listinfo/devel
>>
>>
>>
>
>
___
Devel mailing list
Devel@ovirt.org
http://lists.ovirt.org/mailman/listinfo/devel

Re: [ovirt-devel] [VDSM] Excessive warnings in vdsm log

2016-05-18 Thread Vinzenz Feenstra 

> On May 18, 2016, at 1:31 PM, Piotr Kliczewski  wrote:
> 
> Please see [1][2]. We are seeing build issues for f23 so the build failures 
> are not related.
> I asked Gil from infra to take a look.
+1 :-) 

> 
> Thanks,
> Piotr
> 
> [1] https://gerrit.ovirt.org/#/c/57618/ 
> [2] https://gerrit.ovirt.org/#/c/57623 
> 
> On Wed, May 18, 2016 at 1:29 PM, Vinzenz Feenstra  > wrote:
> 
>> On May 18, 2016, at 1:23 PM, Dan Kenigsberg > > wrote:
>> 
>> On Tue, May 17, 2016 at 08:44:11AM +0200, Piotr Kliczewski wrote:
>>> Nir,
>>> 
>>> The warnings were added to annoy people so we could keep the schema align
>>> with the code.
>>> I think that we should use this opportunity to push fixes instead of
>>> disabling it.
>>> 
>>> Thanks,
>>> Piotr
>>> 
>>> On Mon, May 16, 2016 at 10:09 PM, Nir Soffer >> > wrote:
>>> 
 Hi all,
 
 Since data verification patches were merged, vdsm logs is spammed with
 useless warnings (see bellow).
 
 This spam make it harder to debug vdsm.
 
 Please add configuration variable to enable this warnings, and make
 them disabled by default.
 
 Thanks,
 Nir
 
 
 jsonrpc.Executor/0::WARNING::2016-05-16
 23:05:21,719::schemaapi::140::SchemaCache::(_report_inconsistency)
 Following parameters ['ksmMergeAcrossNodes', 'haStats'] were not
 recognized
 jsonrpc.Executor/0::WARNING::2016-05-16
 23:05:21,720::schemaapi::140::SchemaCache::(_report_inconsistency)
 Parameter cpuUserVdsmd is not float type
 jsonrpc.Executor/0::WARNING::2016-05-16
 23:05:21,720::schemaapi::140::SchemaCache::(_report_inconsistency)
 Parameter rxRate is not float type
>> 
>> I'm officially annoyed ;-)
>> we can stop report rxRate and txRate now, as engine-3.6 computes them on
>> its own.
>> 
>> Marcin, can you drop them from the code and schema?
>> 
 jsonrpc.Executor/0::WARNING::2016-05-16
 23:05:21,720::schemaapi::140::SchemaCache::(_report_inconsistency)
 Parameter cpuLoad is not float type
 jsonrpc.Executor/0::WARNING::2016-05-16
 23:05:21,720::schemaapi::140::SchemaCache::(_report_inconsistency)
 Parameter memUsed is not uint type
 jsonrpc.Executor/0::WARNING::2016-05-16
 23:05:21,720::schemaapi::140::SchemaCache::(_report_inconsistency)
 Parameter cpuIdle is not float type
 jsonrpc.Executor/0::WARNING::2016-05-16
 23:05:21,721::schemaapi::140::SchemaCache::(_report_inconsistency)
 Parameter txRate is not float type
 jsonrpc.Executor/0::WARNING::2016-05-16
 23:05:21,721::schemaapi::140::SchemaCache::(_report_inconsistency)
 Parameter txDropped is not uint type
 jsonrpc.Executor/0::WARNING::2016-05-16
 23:05:21,721::schemaapi::140::SchemaCache::(_report_inconsistency)
 Parameter elapsedTime is not uint type
 jsonrpc.Executor/0::WARNING::2016-05-16
 23:05:21,721::schemaapi::140::SchemaCache::(_report_inconsistency)
 Parameter netConfigDirty is not boolean type
 jsonrpc.Executor/0::WARNING::2016-05-16
 23:05:21,721::schemaapi::140::SchemaCache::(_report_inconsistency)
 Parameter rxErrors is not uint type
>> 
>> Piotr, how can we specify in the schema the very common type of "string
>> that happens to include only unsigned decimal number”?
> 
> I remember that we talked about that with Piotr and that the best solution 
> would be a custom type for that, so that it can be checked
> e.g. struint and strint as types, that means that those are integers 
> (signed/unsigned however passed as string) (Due to historical XMLRPC reasons)
> 
>> 
>> ___
>> Devel mailing list
>> Devel@ovirt.org 
>> http://lists.ovirt.org/mailman/listinfo/devel 
>> 
> 

___
Devel mailing list
Devel@ovirt.org
http://lists.ovirt.org/mailman/listinfo/devel

Re: [ovirt-devel] [VDSM] Excessive warnings in vdsm log

2016-05-18 Thread Piotr Kliczewski 
Please see [1][2]. We are seeing build issues for f23 so the build failures
are not related.
I asked Gil from infra to take a look.

Thanks,
Piotr

[1] https://gerrit.ovirt.org/#/c/57618/
[2] https://gerrit.ovirt.org/#/c/57623

On Wed, May 18, 2016 at 1:29 PM, Vinzenz Feenstra 
wrote:

>
> On May 18, 2016, at 1:23 PM, Dan Kenigsberg  wrote:
>
> On Tue, May 17, 2016 at 08:44:11AM +0200, Piotr Kliczewski wrote:
>
> Nir,
>
> The warnings were added to annoy people so we could keep the schema align
> with the code.
> I think that we should use this opportunity to push fixes instead of
> disabling it.
>
> Thanks,
> Piotr
>
> On Mon, May 16, 2016 at 10:09 PM, Nir Soffer  wrote:
>
> Hi all,
>
> Since data verification patches were merged, vdsm logs is spammed with
> useless warnings (see bellow).
>
> This spam make it harder to debug vdsm.
>
> Please add configuration variable to enable this warnings, and make
> them disabled by default.
>
> Thanks,
> Nir
>
> 
> jsonrpc.Executor/0::WARNING::2016-05-16
> 23:05:21,719::schemaapi::140::SchemaCache::(_report_inconsistency)
> Following parameters ['ksmMergeAcrossNodes', 'haStats'] were not
> recognized
> jsonrpc.Executor/0::WARNING::2016-05-16
> 23:05:21,720::schemaapi::140::SchemaCache::(_report_inconsistency)
> Parameter cpuUserVdsmd is not float type
> jsonrpc.Executor/0::WARNING::2016-05-16
> 23:05:21,720::schemaapi::140::SchemaCache::(_report_inconsistency)
> Parameter rxRate is not float type
>
>
> I'm officially annoyed ;-)
> we can stop report rxRate and txRate now, as engine-3.6 computes them on
> its own.
>
> Marcin, can you drop them from the code and schema?
>
> jsonrpc.Executor/0::WARNING::2016-05-16
> 23:05:21,720::schemaapi::140::SchemaCache::(_report_inconsistency)
> Parameter cpuLoad is not float type
> jsonrpc.Executor/0::WARNING::2016-05-16
> 23:05:21,720::schemaapi::140::SchemaCache::(_report_inconsistency)
> Parameter memUsed is not uint type
> jsonrpc.Executor/0::WARNING::2016-05-16
> 23:05:21,720::schemaapi::140::SchemaCache::(_report_inconsistency)
> Parameter cpuIdle is not float type
> jsonrpc.Executor/0::WARNING::2016-05-16
> 23:05:21,721::schemaapi::140::SchemaCache::(_report_inconsistency)
> Parameter txRate is not float type
> jsonrpc.Executor/0::WARNING::2016-05-16
> 23:05:21,721::schemaapi::140::SchemaCache::(_report_inconsistency)
> Parameter txDropped is not uint type
> jsonrpc.Executor/0::WARNING::2016-05-16
> 23:05:21,721::schemaapi::140::SchemaCache::(_report_inconsistency)
> Parameter elapsedTime is not uint type
> jsonrpc.Executor/0::WARNING::2016-05-16
> 23:05:21,721::schemaapi::140::SchemaCache::(_report_inconsistency)
> Parameter netConfigDirty is not boolean type
> jsonrpc.Executor/0::WARNING::2016-05-16
> 23:05:21,721::schemaapi::140::SchemaCache::(_report_inconsistency)
> Parameter rxErrors is not uint type
>
>
> Piotr, how can we specify in the schema the very common type of "string
> that happens to include only unsigned decimal number”?
>
>
> I remember that we talked about that with Piotr and that the best solution
> would be a custom type for that, so that it can be checked
> e.g. struint and strint as types, that means that those are integers
> (signed/unsigned however passed as string) (Due to historical XMLRPC
> reasons)
>
>
> ___
> Devel mailing list
> Devel@ovirt.org
> http://lists.ovirt.org/mailman/listinfo/devel
>
>
>
___
Devel mailing list
Devel@ovirt.org
http://lists.ovirt.org/mailman/listinfo/devel

Re: [ovirt-devel] [VDSM] Excessive warnings in vdsm log

2016-05-18 Thread Vinzenz Feenstra 

> On May 18, 2016, at 1:23 PM, Dan Kenigsberg  wrote:
> 
> On Tue, May 17, 2016 at 08:44:11AM +0200, Piotr Kliczewski wrote:
>> Nir,
>> 
>> The warnings were added to annoy people so we could keep the schema align
>> with the code.
>> I think that we should use this opportunity to push fixes instead of
>> disabling it.
>> 
>> Thanks,
>> Piotr
>> 
>> On Mon, May 16, 2016 at 10:09 PM, Nir Soffer  wrote:
>> 
>>> Hi all,
>>> 
>>> Since data verification patches were merged, vdsm logs is spammed with
>>> useless warnings (see bellow).
>>> 
>>> This spam make it harder to debug vdsm.
>>> 
>>> Please add configuration variable to enable this warnings, and make
>>> them disabled by default.
>>> 
>>> Thanks,
>>> Nir
>>> 
>>> 
>>> jsonrpc.Executor/0::WARNING::2016-05-16
>>> 23:05:21,719::schemaapi::140::SchemaCache::(_report_inconsistency)
>>> Following parameters ['ksmMergeAcrossNodes', 'haStats'] were not
>>> recognized
>>> jsonrpc.Executor/0::WARNING::2016-05-16
>>> 23:05:21,720::schemaapi::140::SchemaCache::(_report_inconsistency)
>>> Parameter cpuUserVdsmd is not float type
>>> jsonrpc.Executor/0::WARNING::2016-05-16
>>> 23:05:21,720::schemaapi::140::SchemaCache::(_report_inconsistency)
>>> Parameter rxRate is not float type
> 
> I'm officially annoyed ;-)
> we can stop report rxRate and txRate now, as engine-3.6 computes them on
> its own.
> 
> Marcin, can you drop them from the code and schema?
> 
>>> jsonrpc.Executor/0::WARNING::2016-05-16
>>> 23:05:21,720::schemaapi::140::SchemaCache::(_report_inconsistency)
>>> Parameter cpuLoad is not float type
>>> jsonrpc.Executor/0::WARNING::2016-05-16
>>> 23:05:21,720::schemaapi::140::SchemaCache::(_report_inconsistency)
>>> Parameter memUsed is not uint type
>>> jsonrpc.Executor/0::WARNING::2016-05-16
>>> 23:05:21,720::schemaapi::140::SchemaCache::(_report_inconsistency)
>>> Parameter cpuIdle is not float type
>>> jsonrpc.Executor/0::WARNING::2016-05-16
>>> 23:05:21,721::schemaapi::140::SchemaCache::(_report_inconsistency)
>>> Parameter txRate is not float type
>>> jsonrpc.Executor/0::WARNING::2016-05-16
>>> 23:05:21,721::schemaapi::140::SchemaCache::(_report_inconsistency)
>>> Parameter txDropped is not uint type
>>> jsonrpc.Executor/0::WARNING::2016-05-16
>>> 23:05:21,721::schemaapi::140::SchemaCache::(_report_inconsistency)
>>> Parameter elapsedTime is not uint type
>>> jsonrpc.Executor/0::WARNING::2016-05-16
>>> 23:05:21,721::schemaapi::140::SchemaCache::(_report_inconsistency)
>>> Parameter netConfigDirty is not boolean type
>>> jsonrpc.Executor/0::WARNING::2016-05-16
>>> 23:05:21,721::schemaapi::140::SchemaCache::(_report_inconsistency)
>>> Parameter rxErrors is not uint type
> 
> Piotr, how can we specify in the schema the very common type of "string
> that happens to include only unsigned decimal number”?

I remember that we talked about that with Piotr and that the best solution 
would be a custom type for that, so that it can be checked
e.g. struint and strint as types, that means that those are integers 
(signed/unsigned however passed as string) (Due to historical XMLRPC reasons)

> 
> ___
> Devel mailing list
> Devel@ovirt.org 
> http://lists.ovirt.org/mailman/listinfo/devel 
> 
___
Devel mailing list
Devel@ovirt.org
http://lists.ovirt.org/mailman/listinfo/devel

Re: [ovirt-devel] [VDSM] Excessive warnings in vdsm log

2016-05-18 Thread Dan Kenigsberg 
On Tue, May 17, 2016 at 08:44:11AM +0200, Piotr Kliczewski wrote:
> Nir,
> 
> The warnings were added to annoy people so we could keep the schema align
> with the code.
> I think that we should use this opportunity to push fixes instead of
> disabling it.
> 
> Thanks,
> Piotr
> 
> On Mon, May 16, 2016 at 10:09 PM, Nir Soffer  wrote:
> 
> > Hi all,
> >
> > Since data verification patches were merged, vdsm logs is spammed with
> > useless warnings (see bellow).
> >
> > This spam make it harder to debug vdsm.
> >
> > Please add configuration variable to enable this warnings, and make
> > them disabled by default.
> >
> > Thanks,
> > Nir
> >
> > 
> > jsonrpc.Executor/0::WARNING::2016-05-16
> > 23:05:21,719::schemaapi::140::SchemaCache::(_report_inconsistency)
> > Following parameters ['ksmMergeAcrossNodes', 'haStats'] were not
> > recognized
> > jsonrpc.Executor/0::WARNING::2016-05-16
> > 23:05:21,720::schemaapi::140::SchemaCache::(_report_inconsistency)
> > Parameter cpuUserVdsmd is not float type
> > jsonrpc.Executor/0::WARNING::2016-05-16
> > 23:05:21,720::schemaapi::140::SchemaCache::(_report_inconsistency)
> > Parameter rxRate is not float type

I'm officially annoyed ;-)
we can stop report rxRate and txRate now, as engine-3.6 computes them on
its own.

Marcin, can you drop them from the code and schema?

> > jsonrpc.Executor/0::WARNING::2016-05-16
> > 23:05:21,720::schemaapi::140::SchemaCache::(_report_inconsistency)
> > Parameter cpuLoad is not float type
> > jsonrpc.Executor/0::WARNING::2016-05-16
> > 23:05:21,720::schemaapi::140::SchemaCache::(_report_inconsistency)
> > Parameter memUsed is not uint type
> > jsonrpc.Executor/0::WARNING::2016-05-16
> > 23:05:21,720::schemaapi::140::SchemaCache::(_report_inconsistency)
> > Parameter cpuIdle is not float type
> > jsonrpc.Executor/0::WARNING::2016-05-16
> > 23:05:21,721::schemaapi::140::SchemaCache::(_report_inconsistency)
> > Parameter txRate is not float type
> > jsonrpc.Executor/0::WARNING::2016-05-16
> > 23:05:21,721::schemaapi::140::SchemaCache::(_report_inconsistency)
> > Parameter txDropped is not uint type
> > jsonrpc.Executor/0::WARNING::2016-05-16
> > 23:05:21,721::schemaapi::140::SchemaCache::(_report_inconsistency)
> > Parameter elapsedTime is not uint type
> > jsonrpc.Executor/0::WARNING::2016-05-16
> > 23:05:21,721::schemaapi::140::SchemaCache::(_report_inconsistency)
> > Parameter netConfigDirty is not boolean type
> > jsonrpc.Executor/0::WARNING::2016-05-16
> > 23:05:21,721::schemaapi::140::SchemaCache::(_report_inconsistency)
> > Parameter rxErrors is not uint type

Piotr, how can we specify in the schema the very common type of "string
that happens to include only unsigned decimal number"?

___
Devel mailing list
Devel@ovirt.org
http://lists.ovirt.org/mailman/listinfo/devel

Re: [ovirt-devel] [VDSM] Excessive warnings in vdsm log

2016-05-17 Thread Nir Soffer 
The best way is to use warnings.warn instead of logging.warning - we
already using this for similar stuf, and we have a configuration option ti
enabke the warnings in the logs.

Nir
בתאריך 17 במאי 2016 11:33 לפנה״צ,‏ "Martin Sivak"  כתב:

Can't we just configure the logging level for the specific logger?
That is why we use them. Keep the logging, but change the default for
schemaapi to ERROR for now.

Martin

On Tue, May 17, 2016 at 8:47 AM, Yaniv Kaul  wrote:
>
>
> On Tue, May 17, 2016 at 9:44 AM, Piotr Kliczewski 
> wrote:
>>
>> Nir,
>>
>> The warnings were added to annoy people so we could keep the schema align
>> with the code.
>> I think that we should use this opportunity to push fixes instead of
>> disabling it.
>
>
> I agree to the end goal, but I don't think this specific method would
work.
> I much rather we'll do a concentrated effort to drastically reduce those -
> and I don't think right now is the best time for it (a bit late for 4.0)
> Y.
>
>>
>> Thanks,
>> Piotr
>>
>> On Mon, May 16, 2016 at 10:09 PM, Nir Soffer  wrote:
>>>
>>> Hi all,
>>>
>>> Since data verification patches were merged, vdsm logs is spammed with
>>> useless warnings (see bellow).
>>>
>>> This spam make it harder to debug vdsm.
>>>
>>> Please add configuration variable to enable this warnings, and make
>>> them disabled by default.
>>>
>>> Thanks,
>>> Nir
>>>
>>> 
>>> jsonrpc.Executor/0::WARNING::2016-05-16
>>> 23:05:21,719::schemaapi::140::SchemaCache::(_report_inconsistency)
>>> Following parameters ['ksmMergeAcrossNodes', 'haStats'] were not
>>> recognized
>>> jsonrpc.Executor/0::WARNING::2016-05-16
>>> 23:05:21,720::schemaapi::140::SchemaCache::(_report_inconsistency)
>>> Parameter cpuUserVdsmd is not float type
>>> jsonrpc.Executor/0::WARNING::2016-05-16
>>> 23:05:21,720::schemaapi::140::SchemaCache::(_report_inconsistency)
>>> Parameter rxRate is not float type
>>> jsonrpc.Executor/0::WARNING::2016-05-16
>>> 23:05:21,720::schemaapi::140::SchemaCache::(_report_inconsistency)
>>> Parameter cpuLoad is not float type
>>> jsonrpc.Executor/0::WARNING::2016-05-16
>>> 23:05:21,720::schemaapi::140::SchemaCache::(_report_inconsistency)
>>> Parameter memUsed is not uint type
>>> jsonrpc.Executor/0::WARNING::2016-05-16
>>> 23:05:21,720::schemaapi::140::SchemaCache::(_report_inconsistency)
>>> Parameter cpuIdle is not float type
>>> jsonrpc.Executor/0::WARNING::2016-05-16
>>> 23:05:21,721::schemaapi::140::SchemaCache::(_report_inconsistency)
>>> Parameter txRate is not float type
>>> jsonrpc.Executor/0::WARNING::2016-05-16
>>> 23:05:21,721::schemaapi::140::SchemaCache::(_report_inconsistency)
>>> Parameter txDropped is not uint type
>>> jsonrpc.Executor/0::WARNING::2016-05-16
>>> 23:05:21,721::schemaapi::140::SchemaCache::(_report_inconsistency)
>>> Parameter elapsedTime is not uint type
>>> jsonrpc.Executor/0::WARNING::2016-05-16
>>> 23:05:21,721::schemaapi::140::SchemaCache::(_report_inconsistency)
>>> Parameter netConfigDirty is not boolean type
>>> jsonrpc.Executor/0::WARNING::2016-05-16
>>> 23:05:21,721::schemaapi::140::SchemaCache::(_report_inconsistency)
>>> Parameter rxErrors is not uint type
>>> jsonrpc.Executor/0::WARNING::2016-05-16
>>> 23:05:21,721::schemaapi::140::SchemaCache::(_report_inconsistency)
>>> Parameter rxRate is not float type
>>> jsonrpc.Executor/0::WARNING::2016-05-16
>>> 23:05:21,722::schemaapi::140::SchemaCache::(_report_inconsistency)
>>> Parameter rx is not uint type
>>> jsonrpc.Executor/0::WARNING::2016-05-16
>>> 23:05:21,722::schemaapi::140::SchemaCache::(_report_inconsistency)
>>> Parameter txDropped is not uint type
>>> jsonrpc.Executor/0::WARNING::2016-05-16
>>> 23:05:21,722::schemaapi::140::SchemaCache::(_report_inconsistency)
>>> Parameter txErrors is not uint type
>>> jsonrpc.Executor/0::WARNING::2016-05-16
>>> 23:05:21,722::schemaapi::140::SchemaCache::(_report_inconsistency)
>>> Parameter txRate is not float type
>>> jsonrpc.Executor/0::WARNING::2016-05-16
>>> 23:05:21,722::schemaapi::140::SchemaCache::(_report_inconsistency)
>>> Parameter speed is not uint type
>>> jsonrpc.Executor/0::WARNING::2016-05-16
>>> 23:05:21,723::schemaapi::140::SchemaCache::(_report_inconsistency)
>>> Parameter tx is not uint type
>>> jsonrpc.Executor/0::WARNING::2016-05-16
>>> 23:05:21,723::schemaapi::140::SchemaCache::(_report_inconsistency)
>>> Parameter rxDropped is not uint type
>>> jsonrpc.Executor/0::WARNING::2016-05-16
>>> 23:05:21,723::schemaapi::140::SchemaCache::(_report_inconsistency)
>>> Parameter rxErrors is not uint type
>>> jsonrpc.Executor/0::WARNING::2016-05-16
>>> 23:05:21,723::schemaapi::140::SchemaCache::(_report_inconsistency)
>>> Parameter rxRate is not float type
>>> jsonrpc.Executor/0::WARNING::2016-05-16
>>> 23:05:21,723::schemaapi::140::SchemaCache::(_report_inconsistency)
>>> Parameter rx is not uint type
>>> jsonrpc.Executor/0::WARNING::2016-05-16
>>>

Re: [ovirt-devel] [VDSM] Excessive warnings in vdsm log

2016-05-17 Thread Yaniv Kaul 
On Tue, May 17, 2016 at 9:44 AM, Piotr Kliczewski 
wrote:

> Nir,
>
> The warnings were added to annoy people so we could keep the schema align
> with the code.
> I think that we should use this opportunity to push fixes instead of
> disabling it.
>

I agree to the end goal, but I don't think this specific method would work.
I much rather we'll do a concentrated effort to drastically reduce those -
and I don't think right now is the best time for it (a bit late for 4.0)
Y.


> Thanks,
> Piotr
>
> On Mon, May 16, 2016 at 10:09 PM, Nir Soffer  wrote:
>
>> Hi all,
>>
>> Since data verification patches were merged, vdsm logs is spammed with
>> useless warnings (see bellow).
>>
>> This spam make it harder to debug vdsm.
>>
>> Please add configuration variable to enable this warnings, and make
>> them disabled by default.
>>
>> Thanks,
>> Nir
>>
>> 
>> jsonrpc.Executor/0::WARNING::2016-05-16
>> 23:05:21,719::schemaapi::140::SchemaCache::(_report_inconsistency)
>> Following parameters ['ksmMergeAcrossNodes', 'haStats'] were not
>> recognized
>> jsonrpc.Executor/0::WARNING::2016-05-16
>> 23:05:21,720::schemaapi::140::SchemaCache::(_report_inconsistency)
>> Parameter cpuUserVdsmd is not float type
>> jsonrpc.Executor/0::WARNING::2016-05-16
>> 23:05:21,720::schemaapi::140::SchemaCache::(_report_inconsistency)
>> Parameter rxRate is not float type
>> jsonrpc.Executor/0::WARNING::2016-05-16
>> 23:05:21,720::schemaapi::140::SchemaCache::(_report_inconsistency)
>> Parameter cpuLoad is not float type
>> jsonrpc.Executor/0::WARNING::2016-05-16
>> 23:05:21,720::schemaapi::140::SchemaCache::(_report_inconsistency)
>> Parameter memUsed is not uint type
>> jsonrpc.Executor/0::WARNING::2016-05-16
>> 23:05:21,720::schemaapi::140::SchemaCache::(_report_inconsistency)
>> Parameter cpuIdle is not float type
>> jsonrpc.Executor/0::WARNING::2016-05-16
>> 23:05:21,721::schemaapi::140::SchemaCache::(_report_inconsistency)
>> Parameter txRate is not float type
>> jsonrpc.Executor/0::WARNING::2016-05-16
>> 23:05:21,721::schemaapi::140::SchemaCache::(_report_inconsistency)
>> Parameter txDropped is not uint type
>> jsonrpc.Executor/0::WARNING::2016-05-16
>> 23:05:21,721::schemaapi::140::SchemaCache::(_report_inconsistency)
>> Parameter elapsedTime is not uint type
>> jsonrpc.Executor/0::WARNING::2016-05-16
>> 23:05:21,721::schemaapi::140::SchemaCache::(_report_inconsistency)
>> Parameter netConfigDirty is not boolean type
>> jsonrpc.Executor/0::WARNING::2016-05-16
>> 23:05:21,721::schemaapi::140::SchemaCache::(_report_inconsistency)
>> Parameter rxErrors is not uint type
>> jsonrpc.Executor/0::WARNING::2016-05-16
>> 23:05:21,721::schemaapi::140::SchemaCache::(_report_inconsistency)
>> Parameter rxRate is not float type
>> jsonrpc.Executor/0::WARNING::2016-05-16
>> 23:05:21,722::schemaapi::140::SchemaCache::(_report_inconsistency)
>> Parameter rx is not uint type
>> jsonrpc.Executor/0::WARNING::2016-05-16
>> 23:05:21,722::schemaapi::140::SchemaCache::(_report_inconsistency)
>> Parameter txDropped is not uint type
>> jsonrpc.Executor/0::WARNING::2016-05-16
>> 23:05:21,722::schemaapi::140::SchemaCache::(_report_inconsistency)
>> Parameter txErrors is not uint type
>> jsonrpc.Executor/0::WARNING::2016-05-16
>> 23:05:21,722::schemaapi::140::SchemaCache::(_report_inconsistency)
>> Parameter txRate is not float type
>> jsonrpc.Executor/0::WARNING::2016-05-16
>> 23:05:21,722::schemaapi::140::SchemaCache::(_report_inconsistency)
>> Parameter speed is not uint type
>> jsonrpc.Executor/0::WARNING::2016-05-16
>> 23:05:21,723::schemaapi::140::SchemaCache::(_report_inconsistency)
>> Parameter tx is not uint type
>> jsonrpc.Executor/0::WARNING::2016-05-16
>> 23:05:21,723::schemaapi::140::SchemaCache::(_report_inconsistency)
>> Parameter rxDropped is not uint type
>> jsonrpc.Executor/0::WARNING::2016-05-16
>> 23:05:21,723::schemaapi::140::SchemaCache::(_report_inconsistency)
>> Parameter rxErrors is not uint type
>> jsonrpc.Executor/0::WARNING::2016-05-16
>> 23:05:21,723::schemaapi::140::SchemaCache::(_report_inconsistency)
>> Parameter rxRate is not float type
>> jsonrpc.Executor/0::WARNING::2016-05-16
>> 23:05:21,723::schemaapi::140::SchemaCache::(_report_inconsistency)
>> Parameter rx is not uint type
>> jsonrpc.Executor/0::WARNING::2016-05-16
>> 23:05:21,723::schemaapi::140::SchemaCache::(_report_inconsistency)
>> Parameter txDropped is not uint type
>> jsonrpc.Executor/0::WARNING::2016-05-16
>> 23:05:21,724::schemaapi::140::SchemaCache::(_report_inconsistency)
>> Parameter txErrors is not uint type
>> jsonrpc.Executor/0::WARNING::2016-05-16
>> 23:05:21,724::schemaapi::140::SchemaCache::(_report_inconsistency)
>> Parameter txRate is not float type
>> jsonrpc.Executor/0::WARNING::2016-05-16
>> 23:05:21,724::schemaapi::140::SchemaCache::(_report_inconsistency)
>> Parameter speed is not uint type
>> jsonrpc.Executor/0::WARNING::2016-05-16
>> 23:05:21,724::schemaapi::140::SchemaCache::(_report_inconsistency)

Re: [ovirt-devel] [VDSM] Excessive warnings in vdsm log

2016-05-17 Thread Piotr Kliczewski 
Nir,

The warnings were added to annoy people so we could keep the schema align
with the code.
I think that we should use this opportunity to push fixes instead of
disabling it.

Thanks,
Piotr

On Mon, May 16, 2016 at 10:09 PM, Nir Soffer  wrote:

> Hi all,
>
> Since data verification patches were merged, vdsm logs is spammed with
> useless warnings (see bellow).
>
> This spam make it harder to debug vdsm.
>
> Please add configuration variable to enable this warnings, and make
> them disabled by default.
>
> Thanks,
> Nir
>
> 
> jsonrpc.Executor/0::WARNING::2016-05-16
> 23:05:21,719::schemaapi::140::SchemaCache::(_report_inconsistency)
> Following parameters ['ksmMergeAcrossNodes', 'haStats'] were not
> recognized
> jsonrpc.Executor/0::WARNING::2016-05-16
> 23:05:21,720::schemaapi::140::SchemaCache::(_report_inconsistency)
> Parameter cpuUserVdsmd is not float type
> jsonrpc.Executor/0::WARNING::2016-05-16
> 23:05:21,720::schemaapi::140::SchemaCache::(_report_inconsistency)
> Parameter rxRate is not float type
> jsonrpc.Executor/0::WARNING::2016-05-16
> 23:05:21,720::schemaapi::140::SchemaCache::(_report_inconsistency)
> Parameter cpuLoad is not float type
> jsonrpc.Executor/0::WARNING::2016-05-16
> 23:05:21,720::schemaapi::140::SchemaCache::(_report_inconsistency)
> Parameter memUsed is not uint type
> jsonrpc.Executor/0::WARNING::2016-05-16
> 23:05:21,720::schemaapi::140::SchemaCache::(_report_inconsistency)
> Parameter cpuIdle is not float type
> jsonrpc.Executor/0::WARNING::2016-05-16
> 23:05:21,721::schemaapi::140::SchemaCache::(_report_inconsistency)
> Parameter txRate is not float type
> jsonrpc.Executor/0::WARNING::2016-05-16
> 23:05:21,721::schemaapi::140::SchemaCache::(_report_inconsistency)
> Parameter txDropped is not uint type
> jsonrpc.Executor/0::WARNING::2016-05-16
> 23:05:21,721::schemaapi::140::SchemaCache::(_report_inconsistency)
> Parameter elapsedTime is not uint type
> jsonrpc.Executor/0::WARNING::2016-05-16
> 23:05:21,721::schemaapi::140::SchemaCache::(_report_inconsistency)
> Parameter netConfigDirty is not boolean type
> jsonrpc.Executor/0::WARNING::2016-05-16
> 23:05:21,721::schemaapi::140::SchemaCache::(_report_inconsistency)
> Parameter rxErrors is not uint type
> jsonrpc.Executor/0::WARNING::2016-05-16
> 23:05:21,721::schemaapi::140::SchemaCache::(_report_inconsistency)
> Parameter rxRate is not float type
> jsonrpc.Executor/0::WARNING::2016-05-16
> 23:05:21,722::schemaapi::140::SchemaCache::(_report_inconsistency)
> Parameter rx is not uint type
> jsonrpc.Executor/0::WARNING::2016-05-16
> 23:05:21,722::schemaapi::140::SchemaCache::(_report_inconsistency)
> Parameter txDropped is not uint type
> jsonrpc.Executor/0::WARNING::2016-05-16
> 23:05:21,722::schemaapi::140::SchemaCache::(_report_inconsistency)
> Parameter txErrors is not uint type
> jsonrpc.Executor/0::WARNING::2016-05-16
> 23:05:21,722::schemaapi::140::SchemaCache::(_report_inconsistency)
> Parameter txRate is not float type
> jsonrpc.Executor/0::WARNING::2016-05-16
> 23:05:21,722::schemaapi::140::SchemaCache::(_report_inconsistency)
> Parameter speed is not uint type
> jsonrpc.Executor/0::WARNING::2016-05-16
> 23:05:21,723::schemaapi::140::SchemaCache::(_report_inconsistency)
> Parameter tx is not uint type
> jsonrpc.Executor/0::WARNING::2016-05-16
> 23:05:21,723::schemaapi::140::SchemaCache::(_report_inconsistency)
> Parameter rxDropped is not uint type
> jsonrpc.Executor/0::WARNING::2016-05-16
> 23:05:21,723::schemaapi::140::SchemaCache::(_report_inconsistency)
> Parameter rxErrors is not uint type
> jsonrpc.Executor/0::WARNING::2016-05-16
> 23:05:21,723::schemaapi::140::SchemaCache::(_report_inconsistency)
> Parameter rxRate is not float type
> jsonrpc.Executor/0::WARNING::2016-05-16
> 23:05:21,723::schemaapi::140::SchemaCache::(_report_inconsistency)
> Parameter rx is not uint type
> jsonrpc.Executor/0::WARNING::2016-05-16
> 23:05:21,723::schemaapi::140::SchemaCache::(_report_inconsistency)
> Parameter txDropped is not uint type
> jsonrpc.Executor/0::WARNING::2016-05-16
> 23:05:21,724::schemaapi::140::SchemaCache::(_report_inconsistency)
> Parameter txErrors is not uint type
> jsonrpc.Executor/0::WARNING::2016-05-16
> 23:05:21,724::schemaapi::140::SchemaCache::(_report_inconsistency)
> Parameter txRate is not float type
> jsonrpc.Executor/0::WARNING::2016-05-16
> 23:05:21,724::schemaapi::140::SchemaCache::(_report_inconsistency)
> Parameter speed is not uint type
> jsonrpc.Executor/0::WARNING::2016-05-16
> 23:05:21,724::schemaapi::140::SchemaCache::(_report_inconsistency)
> Parameter tx is not uint type
> jsonrpc.Executor/0::WARNING::2016-05-16
> 23:05:21,724::schemaapi::140::SchemaCache::(_report_inconsistency)
> Parameter rxDropped is not uint type
> jsonrpc.Executor/0::WARNING::2016-05-16
> 23:05:21,725::schemaapi::140::SchemaCache::(_report_inconsistency)
> Parameter rxErrors is not uint type
> jsonrpc.Executor/0::WARNING::2016-05-16
>

[ovirt-devel] [VDSM] Excessive warnings in vdsm log

2016-05-16 Thread Nir Soffer 
Hi all,

Since data verification patches were merged, vdsm logs is spammed with
useless warnings (see bellow).

This spam make it harder to debug vdsm.

Please add configuration variable to enable this warnings, and make
them disabled by default.

Thanks,
Nir


jsonrpc.Executor/0::WARNING::2016-05-16
23:05:21,719::schemaapi::140::SchemaCache::(_report_inconsistency)
Following parameters ['ksmMergeAcrossNodes', 'haStats'] were not
recognized
jsonrpc.Executor/0::WARNING::2016-05-16
23:05:21,720::schemaapi::140::SchemaCache::(_report_inconsistency)
Parameter cpuUserVdsmd is not float type
jsonrpc.Executor/0::WARNING::2016-05-16
23:05:21,720::schemaapi::140::SchemaCache::(_report_inconsistency)
Parameter rxRate is not float type
jsonrpc.Executor/0::WARNING::2016-05-16
23:05:21,720::schemaapi::140::SchemaCache::(_report_inconsistency)
Parameter cpuLoad is not float type
jsonrpc.Executor/0::WARNING::2016-05-16
23:05:21,720::schemaapi::140::SchemaCache::(_report_inconsistency)
Parameter memUsed is not uint type
jsonrpc.Executor/0::WARNING::2016-05-16
23:05:21,720::schemaapi::140::SchemaCache::(_report_inconsistency)
Parameter cpuIdle is not float type
jsonrpc.Executor/0::WARNING::2016-05-16
23:05:21,721::schemaapi::140::SchemaCache::(_report_inconsistency)
Parameter txRate is not float type
jsonrpc.Executor/0::WARNING::2016-05-16
23:05:21,721::schemaapi::140::SchemaCache::(_report_inconsistency)
Parameter txDropped is not uint type
jsonrpc.Executor/0::WARNING::2016-05-16
23:05:21,721::schemaapi::140::SchemaCache::(_report_inconsistency)
Parameter elapsedTime is not uint type
jsonrpc.Executor/0::WARNING::2016-05-16
23:05:21,721::schemaapi::140::SchemaCache::(_report_inconsistency)
Parameter netConfigDirty is not boolean type
jsonrpc.Executor/0::WARNING::2016-05-16
23:05:21,721::schemaapi::140::SchemaCache::(_report_inconsistency)
Parameter rxErrors is not uint type
jsonrpc.Executor/0::WARNING::2016-05-16
23:05:21,721::schemaapi::140::SchemaCache::(_report_inconsistency)
Parameter rxRate is not float type
jsonrpc.Executor/0::WARNING::2016-05-16
23:05:21,722::schemaapi::140::SchemaCache::(_report_inconsistency)
Parameter rx is not uint type
jsonrpc.Executor/0::WARNING::2016-05-16
23:05:21,722::schemaapi::140::SchemaCache::(_report_inconsistency)
Parameter txDropped is not uint type
jsonrpc.Executor/0::WARNING::2016-05-16
23:05:21,722::schemaapi::140::SchemaCache::(_report_inconsistency)
Parameter txErrors is not uint type
jsonrpc.Executor/0::WARNING::2016-05-16
23:05:21,722::schemaapi::140::SchemaCache::(_report_inconsistency)
Parameter txRate is not float type
jsonrpc.Executor/0::WARNING::2016-05-16
23:05:21,722::schemaapi::140::SchemaCache::(_report_inconsistency)
Parameter speed is not uint type
jsonrpc.Executor/0::WARNING::2016-05-16
23:05:21,723::schemaapi::140::SchemaCache::(_report_inconsistency)
Parameter tx is not uint type
jsonrpc.Executor/0::WARNING::2016-05-16
23:05:21,723::schemaapi::140::SchemaCache::(_report_inconsistency)
Parameter rxDropped is not uint type
jsonrpc.Executor/0::WARNING::2016-05-16
23:05:21,723::schemaapi::140::SchemaCache::(_report_inconsistency)
Parameter rxErrors is not uint type
jsonrpc.Executor/0::WARNING::2016-05-16
23:05:21,723::schemaapi::140::SchemaCache::(_report_inconsistency)
Parameter rxRate is not float type
jsonrpc.Executor/0::WARNING::2016-05-16
23:05:21,723::schemaapi::140::SchemaCache::(_report_inconsistency)
Parameter rx is not uint type
jsonrpc.Executor/0::WARNING::2016-05-16
23:05:21,723::schemaapi::140::SchemaCache::(_report_inconsistency)
Parameter txDropped is not uint type
jsonrpc.Executor/0::WARNING::2016-05-16
23:05:21,724::schemaapi::140::SchemaCache::(_report_inconsistency)
Parameter txErrors is not uint type
jsonrpc.Executor/0::WARNING::2016-05-16
23:05:21,724::schemaapi::140::SchemaCache::(_report_inconsistency)
Parameter txRate is not float type
jsonrpc.Executor/0::WARNING::2016-05-16
23:05:21,724::schemaapi::140::SchemaCache::(_report_inconsistency)
Parameter speed is not uint type
jsonrpc.Executor/0::WARNING::2016-05-16
23:05:21,724::schemaapi::140::SchemaCache::(_report_inconsistency)
Parameter tx is not uint type
jsonrpc.Executor/0::WARNING::2016-05-16
23:05:21,724::schemaapi::140::SchemaCache::(_report_inconsistency)
Parameter rxDropped is not uint type
jsonrpc.Executor/0::WARNING::2016-05-16
23:05:21,725::schemaapi::140::SchemaCache::(_report_inconsistency)
Parameter rxErrors is not uint type
jsonrpc.Executor/0::WARNING::2016-05-16
23:05:21,725::schemaapi::140::SchemaCache::(_report_inconsistency)
Parameter rxRate is not float type
jsonrpc.Executor/0::WARNING::2016-05-16
23:05:21,725::schemaapi::140::SchemaCache::(_report_inconsistency)
Parameter rx is not uint type
jsonrpc.Executor/0::WARNING::2016-05-16
23:05:21,725::schemaapi::140::SchemaCache::(_report_inconsistency)
Parameter txDropped is not uint type
jsonrpc.Executor/0::WARNING::2016-05-16
23:05:21,725::schemaapi::140::SchemaCache::(_report_inconsistency)
Parameter txErrors is not uint