QQ Reconciliator - switch to Logger
This commit is contained in:
parent
8ea452d54f
commit
d47d7f9160
|
@ -19,6 +19,8 @@
|
||||||
-export([init/1, handle_call/3, handle_cast/2, handle_info/2, terminate/2,
|
-export([init/1, handle_call/3, handle_cast/2, handle_info/2, terminate/2,
|
||||||
code_change/3]).
|
code_change/3]).
|
||||||
|
|
||||||
|
-include_lib("kernel/include/logger.hrl").
|
||||||
|
|
||||||
-define(SERVER, ?MODULE).
|
-define(SERVER, ?MODULE).
|
||||||
-define(DEFAULT_INTERVAL, 60_000*60).
|
-define(DEFAULT_INTERVAL, 60_000*60).
|
||||||
-define(DEFAULT_TRIGGER_INTERVAL, 10_000).
|
-define(DEFAULT_TRIGGER_INTERVAL, 10_000).
|
||||||
|
@ -91,8 +93,7 @@ handle_cast({membership_reconciliation_trigger, _Reason}, #state{enabled = false
|
||||||
{noreply, State, hibernate};
|
{noreply, State, hibernate};
|
||||||
handle_cast({membership_reconciliation_trigger, Reason}, #state{timer_ref = OldRef,
|
handle_cast({membership_reconciliation_trigger, Reason}, #state{timer_ref = OldRef,
|
||||||
trigger_interval = Time} = State) ->
|
trigger_interval = Time} = State) ->
|
||||||
rabbit_log:debug("Quorum Queue membership reconciliation triggered: ~p",
|
?LOG_DEBUG("Quorum Queue membership reconciliation scheduled: ~p", [Reason]),
|
||||||
[Reason]),
|
|
||||||
_ = erlang:cancel_timer(OldRef),
|
_ = erlang:cancel_timer(OldRef),
|
||||||
Ref = erlang:send_after(Time, self(), ?EVAL_MSG),
|
Ref = erlang:send_after(Time, self(), ?EVAL_MSG),
|
||||||
{noreply, State#state{timer_ref = Ref}};
|
{noreply, State#state{timer_ref = Ref}};
|
||||||
|
@ -158,7 +159,7 @@ reconciliate_quorum_members(ExpectedNodes, Running, [Q | LocalLeaders],
|
||||||
end
|
end
|
||||||
else
|
else
|
||||||
{timeout, Reason} ->
|
{timeout, Reason} ->
|
||||||
rabbit_log:debug("Find leader timeout: ~p", [Reason]),
|
?LOG_DEBUG("Find leader timeout: ~p", [Reason]),
|
||||||
ok;
|
ok;
|
||||||
_ ->
|
_ ->
|
||||||
noop
|
noop
|
||||||
|
@ -184,13 +185,13 @@ maybe_add_member(Q, Running, MemberNodes, TargetSize) ->
|
||||||
QName = amqqueue:get_name(Q),
|
QName = amqqueue:get_name(Q),
|
||||||
case rabbit_quorum_queue:add_member(Q, Node) of
|
case rabbit_quorum_queue:add_member(Q, Node) of
|
||||||
ok ->
|
ok ->
|
||||||
rabbit_log:debug(
|
?LOG_DEBUG(
|
||||||
"Added node ~ts as a member to ~ts as "
|
"Added node ~ts as a member to ~ts as "
|
||||||
"the queues target group size(#~w) is not met and "
|
"the queues target group size(#~w) is not met and "
|
||||||
"there are enough new nodes(#~w) in the cluster",
|
"there are enough new nodes(#~w) in the cluster",
|
||||||
[Node, rabbit_misc:rs(QName), TargetSize, length(New)]);
|
[Node, rabbit_misc:rs(QName), TargetSize, length(New)]);
|
||||||
{error, Err} ->
|
{error, Err} ->
|
||||||
rabbit_log:warning(
|
?LOG_WARNING(
|
||||||
"~ts: failed to add member (replica) on node ~w, error: ~w",
|
"~ts: failed to add member (replica) on node ~w, error: ~w",
|
||||||
[rabbit_misc:rs(QName), Node, Err])
|
[rabbit_misc:rs(QName), Node, Err])
|
||||||
end,
|
end,
|
||||||
|
@ -235,12 +236,12 @@ remove_members(Q, [Node | Nodes]) ->
|
||||||
case rabbit_quorum_queue:delete_member(Q, Node) of
|
case rabbit_quorum_queue:delete_member(Q, Node) of
|
||||||
ok ->
|
ok ->
|
||||||
QName = amqqueue:get_name(Q),
|
QName = amqqueue:get_name(Q),
|
||||||
rabbit_log:debug("~ts: Successfully removed member (replica) on node ~w",
|
?LOG_DEBUG("~ts: Successfully removed member (replica) on node ~w",
|
||||||
[rabbit_misc:rs(QName), Node]),
|
[rabbit_misc:rs(QName), Node]),
|
||||||
ok;
|
ok;
|
||||||
{error, Err} ->
|
{error, Err} ->
|
||||||
QName = amqqueue:get_name(Q),
|
QName = amqqueue:get_name(Q),
|
||||||
rabbit_log:warning("~ts: failed to remove member (replica) on node "
|
?LOG_DEBUG("~ts: failed to remove member (replica) on node "
|
||||||
"~w, error: ~w",
|
"~w, error: ~w",
|
||||||
[rabbit_misc:rs(QName), Node, Err])
|
[rabbit_misc:rs(QName), Node, Err])
|
||||||
end,
|
end,
|
||||||
|
|
Loading…
Reference in New Issue