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

[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 <nsoffer at redhat.com> 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"?