Skip to content

Commit

Permalink
Remove rabbitmq_stream_prometheus plugin
Browse files Browse the repository at this point in the history
All these metrics, except publishers & consumers, are handled by
rabbitmq_global_metrics, so we currently have duplicates. As I started
removing these, I realised that tests were written in Java - why not
Erlang? - and they seemed way too complicated for what was needed. After
the new rabbitmq_global_metrics, we are left with 2 metrics, and all the
extra code simply doesn't justify them. I am proposing that we add them to
rabbit_global_counters as gauges. Let's discuss @dcorbacho @acogoluegnes

Signed-off-by: Gerhard Lazu <gerhard@lazu.co.uk>
  • Loading branch information
gerhard committed Jun 22, 2021
1 parent c797125 commit fae836f
Show file tree
Hide file tree
Showing 25 changed files with 0 additions and 2,274 deletions.
1 change: 0 additions & 1 deletion BUILD.bazel
Expand Up @@ -77,7 +77,6 @@ ALL_PLUGINS = [
"//deps/rabbitmq_stomp:bazel_erlang_lib",
"//deps/rabbitmq_stream:bazel_erlang_lib",
"//deps/rabbitmq_stream_management:bazel_erlang_lib",
"//deps/rabbitmq_stream_prometheus:bazel_erlang_lib",
"//deps/rabbitmq_top:bazel_erlang_lib",
"//deps/rabbitmq_tracing:bazel_erlang_lib",
"//deps/rabbitmq_trust_store:bazel_erlang_lib",
Expand Down
56 changes: 0 additions & 56 deletions deps/rabbitmq_stream_prometheus/.gitignore

This file was deleted.

55 changes: 0 additions & 55 deletions deps/rabbitmq_stream_prometheus/BUILD.bazel

This file was deleted.

5 changes: 0 additions & 5 deletions deps/rabbitmq_stream_prometheus/LICENSE

This file was deleted.

0 comments on commit fae836f

Please sign in to comment.