mhhh... this is a bit unfortunate: I obviously did not spot it with the initial commit.
Question now: will we break existing scripts when we fix this? Do we need to add an option to impstats to export numbers as, well, numbers? ;-) Opinions? Rainer El mar, 8 jun 2021 a las 6:21, Yuri Bushmelev via rsyslog (<[email protected]>) escribió: > > Hello! > > Hmm.. from what I see here you're right: > https://github.com/rsyslog/rsyslog/blob/master/runtime/statsobj.c#L500 > > Number is formatted as a string.. And there is no "origin" field as well. > I'd prefer to export this data in the same form as `dynstats` counters are. > E.g.: > > { "name": "_sender_stats", "origin": "runtime", "values": { > "192.168.10.18": 1, "192.168.10.19": 12 } } > > This way it's much easier to reuse existing parsers. > > Or... Ideally we don't need this kind of stats at all because a user can > register a dynstats counter for this (I guess).. but I may miss some > internal details here.. > > P.S. I have personal interest here as I made the rsyslog_exporter in Python > :) > > > On Tue, 8 Jun 2021 at 03:41, John Chivian via rsyslog < > [email protected]> wrote: > > > In version 8.2102 the impstats _sender_stat messages field is being > > reported as a string value, not an integer. > > > > > > {"name":"_sender_stat", "sender":"192.168.10.18", "messages":"0"} > > > > > > This is in contrast to essentially everything else that is reported as > > integers. > > > > > > {"name": "udp-15144-out queue", "origin": "core.queue", "size": 0, > > "enqueued": 0, "full": 0, "discarded.full": 0, "discarded.nf": 0, > > "maxqsize": 4 } > > > > > > Config is generated from blow directive… > > > > > > module(load="impstats" > > interval="60" > > resetCounters="on" > > format="json" > > severity="7" > > log.file="/var/log/impstats/impstats.json” > > ) > > > > > > I rather doubt this is intended, and it’s trivial to work around, but the > > maintainers would need to say for sure. I can enter an issue if desired. > > > > Thanks, > > > > _______________________________________________ > > rsyslog mailing list > > https://lists.adiscon.net/mailman/listinfo/rsyslog > > http://www.rsyslog.com/professional-services/ > > What's up with rsyslog? Follow https://twitter.com/rgerhards > > NOTE WELL: This is a PUBLIC mailing list, posts are ARCHIVED by a myriad > > of sites beyond our control. PLEASE UNSUBSCRIBE and DO NOT POST if you > > DON'T LIKE THAT. > > > > -- > Yury Bushmelev > _______________________________________________ > rsyslog mailing list > https://lists.adiscon.net/mailman/listinfo/rsyslog > http://www.rsyslog.com/professional-services/ > What's up with rsyslog? Follow https://twitter.com/rgerhards > NOTE WELL: This is a PUBLIC mailing list, posts are ARCHIVED by a myriad of > sites beyond our control. PLEASE UNSUBSCRIBE and DO NOT POST if you DON'T > LIKE THAT. _______________________________________________ rsyslog mailing list https://lists.adiscon.net/mailman/listinfo/rsyslog http://www.rsyslog.com/professional-services/ What's up with rsyslog? Follow https://twitter.com/rgerhards NOTE WELL: This is a PUBLIC mailing list, posts are ARCHIVED by a myriad of sites beyond our control. PLEASE UNSUBSCRIBE and DO NOT POST if you DON'T LIKE THAT.
