Skip to content

Commit 932a971

Browse files
committed
rabbit_feature_flags: Reset registry after copying feature states
[Why] The copy is likely to install a different list of enabled feature flags compared to the possibly loaded registry. Therefore it needs to be reset. [How] This is already the case, but the reset is done from the clustering code and not right after the copy, leaving a time frame where the feature states are inconsistent between the loaded registry and the on-disk record. This should be the responsibility of the Feature flags subsystem anyway.
1 parent d6bad82 commit 932a971

File tree

1 file changed

+1
-0
lines changed

1 file changed

+1
-0
lines changed

deps/rabbit/src/rabbit_feature_flags.erl

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1331,6 +1331,7 @@ copy_feature_states_after_reset(RemoteNode) ->
13311331
#{domain => ?RMQLOG_DOMAIN_FEAT_FLAGS}),
13321332
case do_write_enabled_feature_flags_list(EnabledFeatureNames) of
13331333
ok ->
1334+
ok = reset_registry(),
13341335
ok;
13351336
{error, Reason} ->
13361337
File = enabled_feature_flags_list_file(),

0 commit comments

Comments
 (0)