Skip to content

Commit c52ff05

Browse files
authored
Relax data path deprecations from critical to warn (#85952)
The deprecations for index and shared data paths were defined at critical level, but these are not removed in 8.0. This commit relaxes these deprecation messages so they are just warnings. relates #85695
1 parent 3fd142f commit c52ff05

File tree

5 files changed

+9
-4
lines changed

5 files changed

+9
-4
lines changed

docs/changelog/85952.yaml

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
pr: 85952
2+
summary: Relax data path deprecations from critical to warn
3+
area: Infra/Core
4+
type: bug
5+
issues: []

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

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -76,7 +76,7 @@ static DeprecationIssue checkIndexDataPath(IndexMetadata indexMetadata) {
7676
final String url = "https://www.elastic.co/guide/en/elasticsearch/reference/7.13/"
7777
+ "breaking-changes-7.13.html#deprecate-shared-data-path-setting";
7878
final String details = "Found index data path configured. Discontinue use of this setting.";
79-
return new DeprecationIssue(DeprecationIssue.Level.CRITICAL, message, url, details, false, null);
79+
return new DeprecationIssue(DeprecationIssue.Level.WARNING, message, url, details, false, null);
8080
}
8181
return null;
8282
}

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

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -129,7 +129,7 @@ static DeprecationIssue checkSharedDataPathSetting(
129129
final String url = "https://www.elastic.co/guide/en/elasticsearch/reference/7.13/"
130130
+ "breaking-changes-7.13.html#deprecate-shared-data-path-setting";
131131
final String details = "Found shared data path configured. Discontinue use of this setting.";
132-
return new DeprecationIssue(DeprecationIssue.Level.CRITICAL, message, url, details, false, null);
132+
return new DeprecationIssue(DeprecationIssue.Level.WARNING, message, url, details, false, null);
133133
}
134134
return null;
135135
}

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

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -93,7 +93,7 @@ public void testIndexDataPathSetting() {
9393
issues,
9494
contains(
9595
new DeprecationIssue(
96-
DeprecationIssue.Level.CRITICAL,
96+
DeprecationIssue.Level.WARNING,
9797
"setting [index.data_path] is deprecated and will be removed in a future version",
9898
expectedUrl,
9999
"Found index data path configured. Discontinue use of this setting.",

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

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -91,7 +91,7 @@ public void testSharedDataPathSetting() {
9191
issues,
9292
contains(
9393
new DeprecationIssue(
94-
DeprecationIssue.Level.CRITICAL,
94+
DeprecationIssue.Level.WARNING,
9595
"setting [path.shared_data] is deprecated and will be removed in a future version",
9696
expectedUrl,
9797
"Found shared data path configured. Discontinue use of this setting.",

0 commit comments

Comments
 (0)