diff options
| author | Michael Klishin <michael@clojurewerkz.org> | 2020-06-05 00:32:43 +0300 |
|---|---|---|
| committer | Michael Klishin <michael@clojurewerkz.org> | 2020-06-05 00:51:51 +0300 |
| commit | a5ff64b9af340b50c2fcdef469b163e88bae86de (patch) | |
| tree | 7c8a8c7a36bb5cee0a59c795222827b0f6765a85 /src | |
| parent | 6b0caa24bcf2ab3dd58d9f887b2363d28c185856 (diff) | |
| download | rabbitmq-server-git-a5ff64b9af340b50c2fcdef469b163e88bae86de.tar.gz | |
Deprecate 'ctl node_health_check'
* It requires a fully booted node, so not generally suitable for a Kubernetes readiness probe.
* It can produce false positives
* It is too intrusive and CPU-intensive to use at scale
* Most operators do not understand what it really does and when they learn about it,
consider it to be too opinionated and intrusive
Time for the One True Health Check™ to retire from duty.
Part of rabbitmq/rabbitmq-cli#426
Diffstat (limited to 'src')
| -rw-r--r-- | src/rabbit_health_check.erl | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/src/rabbit_health_check.erl b/src/rabbit_health_check.erl index d384354dc8..f41c1e6e1b 100644 --- a/src/rabbit_health_check.erl +++ b/src/rabbit_health_check.erl @@ -36,6 +36,8 @@ node(Node, Timeout) -> -spec local() -> ok | {error_string, string()}. local() -> + rabbit_log:warning("rabbitmqctl node_health_check and its HTTP API counterpart are DEPRECATED. " + "See https://www.rabbitmq.com/monitoring.html#health-checks for replacement options."), run_checks([list_channels, list_queues, alarms, rabbit_node_monitor]). %%---------------------------------------------------------------------------- |