Skip to content

Commit 1ac783e

Browse files
Merge pull request #13155 from rabbitmq/mergify/bp/v4.0.x/pr-13154
Log incorrectly claims the limit is per node, but the component count… (backport #13154)
2 parents eeb5cb5 + 9d3728a commit 1ac783e

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

deps/rabbit/src/rabbit_runtime_parameters.erl

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -166,10 +166,10 @@ is_within_limit(Component) ->
166166
case Limit < 0 orelse count_component(Component) < Limit of
167167
true -> ok;
168168
false ->
169-
ErrorMsg = "Limit reached: component ~ts is limited to ~tp per node",
169+
ErrorMsg = "Limit reached: component ~ts is limited to ~tp",
170170
ErrorArgs = [Component, Limit],
171171
rabbit_log:error(ErrorMsg, ErrorArgs),
172-
{errors, [{"component ~ts is limited to ~tp per node", [Component, Limit]}]}
172+
{errors, [{"component ~ts is limited to ~tp", [Component, Limit]}]}
173173
end.
174174

175175
count_component(Component) -> length(list_component(Component)).

deps/rabbit/test/runtime_parameters_SUITE.erl

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -55,7 +55,7 @@ test_limits(Config) ->
5555
test_limits1(_Config) ->
5656
dummy_runtime_parameters:register(),
5757
application:set_env(rabbit, runtime_parameters, [{limits, [{<<"test">>, 1}]}]),
58-
E = {error_string, "Validation failed\n\ncomponent test is limited to 1 per node\n"},
58+
E = {error_string, "Validation failed\n\ncomponent test is limited to 1\n"},
5959
ok = rabbit_runtime_parameters:set_any(<<"/">>, <<"test">>, <<"good">>, <<"">>, none),
6060
E = rabbit_runtime_parameters:set_any(<<"/">>, <<"test">>, <<"good">>, <<"">>, none),
6161
dummy_runtime_parameters:unregister().

0 commit comments

Comments
 (0)