Naming #11304

'user_provided_plugins_data_dir' make it immediately
obvious that the data directory is for 3rd party
plugins (user-provided plugins), and it is
not set up or used internally by RabbitMQ.

Author: michaelklishin

deps/rabbit/src/rabbit_mnesia.erl

@@ -1067,7 +1067,7 @@ mnesia_and_msg_store_files() ->
              rabbit_node_monitor:quorum_filename(),
              rabbit_feature_flags:enabled_feature_flags_list_file(),
              rabbit_khepri:dir(),
-             rabbit_plugins:plugins_data_dir()],
+             rabbit_plugins:user_provided_plugins_data_dir()],
             IgnoredFiles = [filename:basename(File) || File <- IgnoredFiles0],
             ?LOG_DEBUG("Files and directories found in node's data directory: ~ts, of them to be ignored: ~ts",
                             [string:join(lists:usort(List0), ", "), string:join(lists:usort(IgnoredFiles), ", ")]),

deps/rabbit/src/rabbit_plugins.erl

@@ -13,7 +13,7 @@
 -export([validate_plugins/1, format_invalid_plugins/1]).
 -export([is_strictly_plugin/1, strictly_plugins/2, strictly_plugins/1]).
 -export([plugins_dir/0, plugin_names/1, plugins_expand_dir/0, enabled_plugins_file/0]).
--export([plugins_data_dir/0]).
+-export([user_provided_plugins_data_dir/0]).
 -export([is_enabled/1, is_enabled_on_node/2, enabled_plugins/0]).
 -export([which_plugin/1]).
 
@@ -94,13 +94,13 @@ plugins_dir() ->
             filename:join([rabbit:data_dir(), "plugins_dir_stub"])
     end.
 
-plugins_data_dir() ->
-    case application:get_env(rabbit, plugins_data_dir) of
+user_provided_plugins_data_dir() ->
+    case application:get_env(rabbit, user_provided_plugins_data_dir) of
         {ok, PluginsDataDir} ->
             PluginsDataDir;
         _ ->
             DataDir = rabbit_mnesia:dir(),
-            filename:join([DataDir, "plugins_data_dir"])
+            filename:join([DataDir, "user_provided_plugins_data_dir"])
     end.
 
 -spec enabled_plugins_file() -> file:filename().

deps/rabbit/test/metadata_store_migration_SUITE.erl

@@ -124,7 +124,7 @@ from_mnesia_to_khepri(Config) ->
     Maintenance = rabbit_ct_broker_helpers:rpc(Config, 0, rabbit_db_maintenance, get, [Server]),
     ?assertNot(undefined == Maintenance),
 
-    PluginsDataDir = rabbit_ct_broker_helpers:rpc(Config, 0, rabbit_plugins, plugins_data_dir, []),
+    PluginsDataDir = rabbit_ct_broker_helpers:rpc(Config, 0, rabbit_plugins, user_provided_plugins_data_dir, []),
     ok = file:make_dir(PluginsDataDir),
     PluginDataFile = filename:join(PluginsDataDir, "test.txt"),
     ok = file:write_file(PluginDataFile, "test content"),