Skip to content

Commit 39b3010

Browse files
authored
Add node local storage deprecation check (#54383)
The node.local_storage setting has been deprecated and will be removed in 8.0.0. This commit adds a deprecation check to 7.x.
1 parent 67b9b68 commit 39b3010

File tree

3 files changed

+26
-1
lines changed

3 files changed

+26
-1
lines changed

x-pack/plugin/deprecation/src/main/java/org/elasticsearch/xpack/deprecation/DeprecationChecks.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -48,7 +48,8 @@ private DeprecationChecks() {
4848
NodeDeprecationChecks::checkUniqueRealmOrders,
4949
(settings, pluginsAndModules) -> NodeDeprecationChecks.checkThreadPoolListenerQueueSize(settings),
5050
(settings, pluginsAndModules) -> NodeDeprecationChecks.checkThreadPoolListenerSize(settings),
51-
NodeDeprecationChecks::checkClusterRemoteConnectSetting
51+
NodeDeprecationChecks::checkClusterRemoteConnectSetting,
52+
NodeDeprecationChecks::checkNodeLocalStorageSetting
5253
));
5354

5455
static List<Function<IndexMetaData, DeprecationIssue>> INDEX_SETTINGS_CHECKS =

x-pack/plugin/deprecation/src/main/java/org/elasticsearch/xpack/deprecation/NodeDeprecationChecks.java

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -129,6 +129,14 @@ public static DeprecationIssue checkClusterRemoteConnectSetting(final Settings s
129129
);
130130
}
131131

132+
public static DeprecationIssue checkNodeLocalStorageSetting(final Settings settings, final PluginsAndModules pluginsAndModules) {
133+
return checkRemovedSetting(
134+
settings,
135+
Node.NODE_LOCAL_STORAGE_SETTING,
136+
"https://www.elastic.co/guide/en/elasticsearch/reference/7.8/breaking-changes-7.8.html#deprecate-node-local-storage"
137+
);
138+
}
139+
132140
private static DeprecationIssue checkDeprecatedSetting(
133141
final Settings settings,
134142
final PluginsAndModules pluginsAndModules,

x-pack/plugin/deprecation/src/test/java/org/elasticsearch/xpack/deprecation/NodeDeprecationChecksTests.java

Lines changed: 16 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -199,6 +199,22 @@ public void testClusterRemoteConnectSetting() {
199199
assertSettingDeprecationsAndWarnings(new Setting<?>[]{RemoteClusterService.ENABLE_REMOTE_CLUSTERS});
200200
}
201201

202+
public void testNodeLocalStorageSetting() {
203+
final boolean value = randomBoolean();
204+
final Settings settings = Settings.builder().put(Node.NODE_LOCAL_STORAGE_SETTING.getKey(), value).build();
205+
final PluginsAndModules pluginsAndModules = new PluginsAndModules(Collections.emptyList(), Collections.emptyList());
206+
final List<DeprecationIssue> issues =
207+
DeprecationChecks.filterChecks(DeprecationChecks.NODE_SETTINGS_CHECKS, c -> c.apply(settings, pluginsAndModules));
208+
final DeprecationIssue expected = new DeprecationIssue(
209+
DeprecationIssue.Level.CRITICAL,
210+
"setting [node.local_storage] is deprecated and will be removed in the next major version",
211+
"https://www.elastic.co/guide/en/elasticsearch/reference/7.8/breaking-changes-7.8.html#deprecate-node-local-storage",
212+
"the setting [node.local_storage] is currently set to [" + value + "], remove this setting"
213+
);
214+
assertThat(issues, contains(expected));
215+
assertSettingDeprecationsAndWarnings(new Setting<?>[]{Node.NODE_LOCAL_STORAGE_SETTING});
216+
}
217+
202218
public void testRemovedSettingNotSet() {
203219
final Settings settings = Settings.EMPTY;
204220
final Setting<?> removedSetting = Setting.simpleString("node.removed_setting");

0 commit comments

Comments
 (0)