Rounding issue with moving averages

[codingconcepts]

Hi guys,

I'm sure you're already aware of this (so apologies if I'm the 1,000th person to report this):

The weird rounding can affect the Ready, Unacked and Total values.

Cheers,

Rob

[michaelklishin]

What version do you run?

[codingconcepts]

Ah yes, that'd help! I'm running 3.5.0 with Erlang 17.5. I can see that you guys have fixed a number of bugs in 3.5.1 but I can't find a release note with a list of the bugs.

Cheers Michael,

Rob

[dumbbell]

We now put the release notes on GitHub, in the rabbitmq-server project:
https://github.com/rabbitmq/rabbitmq-server/releases/tag/rabbitmq_v3_5_1

They are linked from the website as usual.

[michaelklishin]

@bassrob I'm pretty sure this particular issue wasn't recently fixed. There were related things fixed for 3.5.2 but not this one.

Can you tell us a bit about your environment? What does your workload look like? Was there something suspicious in the log files around the time you've noticed this? Were there any stats DB or node failures around that time?

Thank you.

[codingconcepts]

Hiya, sorry for the delay, I didn't see the notification.

I haven't been monitoring the DB but can tell you it's running up to ~900mps 24/7. Further to that, there's not much more I can offer. It's really nothing serious as we rarely need the Management Console.

Cheers,

Rob

[Gsantomaggio]

To reproduce the bug you have to:

• Select "Moving average time series" (as the image):

• Execute this gist

We're going to continue looking at this issue next week.

[michaelklishin]

@Gsantomaggio good job finding a way to reproduce!

[Gsantomaggio]

Thank you @michaelklishin
the problem is that function fmt_num_thousands https://github.com/rabbitmq/rabbitmq-management/blob/master/priv/www/js/formatters.js#L211 doesn't accept number with point (like this 1234.5678)

In most cases it works. We are going to fix.

[michaelklishin]

@Gsantomaggio 👍

[dumbbell]

Fixed by #60.