Skip to content
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 some dead code and configuration options #12140

Merged
merged 6 commits into from
Aug 29, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
9 changes: 0 additions & 9 deletions deps/rabbit/BUILD.bazel
Original file line number Diff line number Diff line change
Expand Up @@ -35,9 +35,7 @@ _APP_ENV = """[
{num_ssl_acceptors, 10},
{ssl_options, []},
{vm_memory_high_watermark, 0.4},
{vm_memory_high_watermark_paging_ratio, 0.5},
{vm_memory_calculation_strategy, rss},
{memory_monitor_interval, 2500},
{disk_free_limit, 50000000}, %% 50MB
{backing_queue_module, rabbit_variable_queue},
%% 0 ("no limit") would make a better default, but that
Expand Down Expand Up @@ -84,10 +82,7 @@ _APP_ENV = """[
{linger, {true, 0}},
{exit_on_close, false}
]},
{halt_on_upgrade_failure, true},
{ssl_apps, [asn1, crypto, public_key, ssl]},
%% classic queue storage implementation version
{classic_queue_default_version, 2},
%% see rabbitmq-server#227 and related tickets.
%% msg_store_credit_disc_bound only takes effect when
%% messages are persisted to the message store. If messages
Expand All @@ -106,10 +101,6 @@ _APP_ENV = """[
%% and rabbitmq-server#667
{channel_operation_timeout, 15000},

%% see rabbitmq-server#486
{autocluster,
[{peer_discovery_backend, rabbit_peer_discovery_classic_config}]
},
%% used by rabbit_peer_discovery_classic_config
{cluster_nodes, {[], disc}},

Expand Down
9 changes: 0 additions & 9 deletions deps/rabbit/Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -15,9 +15,7 @@ define PROJECT_ENV
{num_ssl_acceptors, 10},
{ssl_options, []},
{vm_memory_high_watermark, 0.4},
{vm_memory_high_watermark_paging_ratio, 0.5},
{vm_memory_calculation_strategy, rss},
{memory_monitor_interval, 2500},
{disk_free_limit, 50000000}, %% 50MB
{backing_queue_module, rabbit_variable_queue},
%% 0 ("no limit") would make a better default, but that
Expand Down Expand Up @@ -67,10 +65,7 @@ define PROJECT_ENV
{linger, {true, 0}},
{exit_on_close, false}
]},
{halt_on_upgrade_failure, true},
{ssl_apps, [asn1, crypto, public_key, ssl]},
%% classic queue storage implementation version
{classic_queue_default_version, 2},
%% see rabbitmq-server#227 and related tickets.
%% msg_store_credit_disc_bound only takes effect when
%% messages are persisted to the message store. If messages
Expand All @@ -92,10 +87,6 @@ define PROJECT_ENV
%% 30 minutes
{consumer_timeout, 1800000},

%% see rabbitmq-server#486
{autocluster,
[{peer_discovery_backend, rabbit_peer_discovery_classic_config}]
},
%% used by rabbit_peer_discovery_classic_config
{cluster_nodes, {[], disc}},

Expand Down
22 changes: 0 additions & 22 deletions deps/rabbit/docs/rabbitmq.conf.example
Original file line number Diff line number Diff line change
Expand Up @@ -404,33 +404,11 @@



## Fraction of the high watermark limit at which queues start to
## page message out to disc in order to free up memory.
## For example, when vm_memory_high_watermark is set to 0.4 and this value is set to 0.5,
## paging can begin as early as when 20% of total available RAM is used by the node.
##
## Values greater than 1.0 can be dangerous and should be used carefully.
##
## One alternative to this is to use durable queues and publish messages
## as persistent (delivery mode = 2). With this combination queues will
## move messages to disk much more rapidly.
##
## Another alternative is to configure queues to page all messages (both
## persistent and transient) to disk as quickly
## as possible, see https://www.rabbitmq.com/docs/lazy-queues.
##
# vm_memory_high_watermark_paging_ratio = 0.5

## Selects Erlang VM memory consumption calculation strategy. Can be `allocated`, `rss` or `legacy` (aliased as `erlang`),
## Introduced in 3.6.11. `rss` is the default as of 3.6.12.
## See https://github.com/rabbitmq/rabbitmq-server/issues/1223 and rabbitmq/rabbitmq-common#224 for background.
# vm_memory_calculation_strategy = rss

## Interval (in milliseconds) at which we perform the check of the memory
## levels against the watermarks.
##
# memory_monitor_interval = 2500

## The total memory available can be calculated from the OS resources
## - default option - or provided as a configuration parameter.
# total_memory_available_override_value = 2GB
Expand Down
5 changes: 5 additions & 0 deletions deps/rabbit/priv/schema/rabbit.schema
Original file line number Diff line number Diff line change
Expand Up @@ -1149,6 +1149,8 @@ fun(Conf) ->
end
end}.

%% DEPRECATED. Not used since RabbitMQ 4.0
%%
%% Fraction of the high watermark limit at which queues start to
%% page message out to disc in order to free up memory.
%%
Expand All @@ -1160,6 +1162,8 @@ end}.
"rabbit.vm_memory_high_watermark_paging_ratio",
[{datatype, float}, {validators, ["less_than_1"]}]}.

%% DEPRECATED. Not used since RabbitMQ 4.0
%%
%% Interval (in milliseconds) at which we perform the check of the memory
%% levels against the watermarks.
%%
Expand Down Expand Up @@ -2554,6 +2558,7 @@ end}.
%% Backing queue version
%%

%% DEPRECATED. Not used since RabbitMQ 4.0
{mapping, "classic_queue.default_version", "rabbit.classic_queue_default_version", [
{datatype, integer},
{validators, ["non_zero_positive_integer"]}
Expand Down
2 changes: 2 additions & 0 deletions deps/rabbit/test/config_schema_SUITE_data/rabbit.snippets
Original file line number Diff line number Diff line change
Expand Up @@ -372,13 +372,15 @@ tcp_listen_options.exit_on_close = false",
vm_memory_high_watermark.relative = 0.4",
[{rabbit,[{vm_memory_high_watermark,{absolute,1073741824}}]}],
[]},
%% DEPRECATED; just for backwards compatibility
{vm_memory_watermark_paging_ratio,
"vm_memory_high_watermark_paging_ratio = 0.75
vm_memory_high_watermark.relative = 0.4",
[{rabbit,
[{vm_memory_high_watermark_paging_ratio,0.75},
{vm_memory_high_watermark,0.4}]}],
[]},
%% DEPRECATED; just for backwards compatibility
{memory_monitor_interval, "memory_monitor_interval = 5000",
[{rabbit,
[{memory_monitor_interval, 5000}]}],
Expand Down

This file was deleted.

3 changes: 1 addition & 2 deletions deps/rabbitmq_prometheus/docker/docker-entrypoint.sh
Original file line number Diff line number Diff line change
Expand Up @@ -55,7 +55,6 @@ rabbitConfigKeys=(
default_pass
default_user
default_vhost
hipe_compile
vm_memory_high_watermark
)
fileConfigKeys=(
Expand Down Expand Up @@ -267,7 +266,7 @@ rabbit_env_config() {
local val="${!var:-}"
local rawVal="$val"
case "$conf" in
fail_if_no_peer_cert|hipe_compile)
fail_if_no_peer_cert)
case "${val,,}" in
false|no|0|'') rawVal='false' ;;
true|yes|1|*) rawVal='true' ;;
Expand Down
1 change: 0 additions & 1 deletion deps/rabbitmq_prometheus/docker/rabbitmq-dist-metrics.conf
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,6 @@ management.listener.port = 15672
management.listener.ssl = false

vm_memory_high_watermark.absolute = 768MiB
vm_memory_high_watermark_paging_ratio = 0.2

cluster_name = rabbitmq-dist-metrics

Expand Down
1 change: 0 additions & 1 deletion deps/rabbitmq_prometheus/docker/rabbitmq-dist-tls.conf
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,6 @@ management.listener.port = 15672
management.listener.ssl = false

vm_memory_high_watermark.absolute = 4GiB
vm_memory_high_watermark_paging_ratio = 0.9
disk_free_limit.absolute = 2048MiB

cluster_name = rabbitmq-dist-tls
Expand Down
1 change: 0 additions & 1 deletion deps/rabbitmq_prometheus/docker/rabbitmq-overview.conf
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,6 @@ management.listener.port = 15672
management.listener.ssl = false

vm_memory_high_watermark.absolute = 768MiB
vm_memory_high_watermark_paging_ratio = 0.2

cluster_name = rabbitmq-overview

Expand Down
4 changes: 1 addition & 3 deletions packaging/docker-image/Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -146,7 +146,6 @@ RUN set -eux; \
--prefix="$ERLANG_INSTALL_PATH_PREFIX" \
--host="$hostArch" \
--build="$buildArch" \
--disable-hipe \
--disable-sctp \
--disable-silent-rules \
--enable-builtin-zlib \
Expand All @@ -166,7 +165,6 @@ RUN set -eux; \
--without-et \
--without-eunit \
--without-ftp \
--without-hipe \
--without-jinterface \
--without-megaco \
--without-observer \
Expand Down Expand Up @@ -329,4 +327,4 @@ RUN set eux; \
rm -rf /var/lib/apt/lists/*; \
rabbitmqadmin --version

EXPOSE 15671 15672
EXPOSE 15671 15672
Loading