-
Notifications
You must be signed in to change notification settings - Fork 80
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Remove zeroed sockets_used/sockets_total #339
base: master
Are you sure you want to change the base?
Conversation
Values are zeroed following 4.x changes.
830e40d
to
b0157fe
Compare
@lhoguin EasyNetQ.Management.Client supports RabbitMQ back to I don't know if there is any rule or commitment about that (@Pliner @zidad, but it's one of the most important things to be written in README), but at least we run tests on Is it ok to make your changes for all those RabbitMQ versions? |
The variables that are removed in this PR can be obtained with better accuracy from system metrics. They are also not very useful but it's possible that someone relies on them. There's no urgency in merging this, the values will just be 0 for 4.x onward. |
@inikulshin dropping these metrics should not break anything even for 3.10 (that has been out of any kind of support from the RabbitMQ Team since Dec 2023), unless the JSON parser/mapper used explicitly requires all keys in a JSON document to be mapped. Which is usually not the case. These metrics are kernel ones and should not be monitored by RabbitMQ itself, hence they were removed from RabbitMQ itself. All that means is that certain API responses now have two fewer JSON object keys. |
FTR, the two metrics were removed in rabbitmq/rabbitmq-server#11436. |
@michaelklishin I'm 99.9% sure nobody ever used those metrics. :) But my question is if they were set in response JSON in any of currently supported RabbitMQ versions? |
They are still returned by 3.13.x and all the series you mention above. |
Values are zeroed following 4.x changes.
FYI I am not a .NET developer so not sure if I broke something.
I just work on rabbit.
There are other values that have been zeroed in recent years,
including many values in [1] that I may send PRs for at a later
time.
[1] https://github.com/EasyNetQ/EasyNetQ.Management.Client/blob/master/Source/EasyNetQ.Management.Client/Model/BackingQueueStatus.cs