Skip to content

Commit 00eec59

Browse files
committed
File Discovery: Remove fallback with zen discovery (#26667)
When adding file based discovery, we added a fallback when the discovery type was set to zen (the default, so everyone got this warning). This commit removes the fallback for 6.0. Setting file discovery should now happen explicitly through the hosts_provider setting. closes #26661
1 parent a9d9e12 commit 00eec59

File tree

3 files changed

+1
-63
lines changed

3 files changed

+1
-63
lines changed

plugins/discovery-file/build.gradle

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -52,6 +52,7 @@ setupSeedNodeAndUnicastHostsFile.doLast {
5252
integTestCluster {
5353
dependsOn setupSeedNodeAndUnicastHostsFile
5454
clusterName = 'discovery-file-test-cluster'
55+
setting 'discovery.zen.hosts_provider', 'file'
5556
extraConfigFile 'discovery-file/unicast_hosts.txt', srcUnicastHostsFile
5657
}
5758

plugins/discovery-file/src/main/java/org/elasticsearch/discovery/file/FileBasedDiscoveryPlugin.java

Lines changed: 0 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -102,20 +102,4 @@ public Map<String, Supplier<UnicastHostsProvider>> getZenHostsProviders(Transpor
102102
() -> new FileBasedUnicastHostsProvider(
103103
new Environment(settings, configPath), transportService, fileBasedDiscoveryExecutorService));
104104
}
105-
106-
@Override
107-
public Settings additionalSettings() {
108-
// For 5.0, the hosts provider was "zen", but this was before the discovery.zen.hosts_provider
109-
// setting existed. This check looks for the legacy zen, and sets the file hosts provider if not set
110-
String discoveryType = DiscoveryModule.DISCOVERY_TYPE_SETTING.get(settings);
111-
if (discoveryType.equals("zen")) {
112-
deprecationLogger.deprecated("Using " + DiscoveryModule.DISCOVERY_TYPE_SETTING.getKey() +
113-
" setting to set hosts provider is deprecated. " +
114-
"Set \"" + DiscoveryModule.DISCOVERY_HOSTS_PROVIDER_SETTING.getKey() + ": file\" instead");
115-
if (DiscoveryModule.DISCOVERY_HOSTS_PROVIDER_SETTING.exists(settings) == false) {
116-
return Settings.builder().put(DiscoveryModule.DISCOVERY_HOSTS_PROVIDER_SETTING.getKey(), "file").build();
117-
}
118-
}
119-
return Settings.EMPTY;
120-
}
121105
}

plugins/discovery-file/src/test/java/org/elasticsearch/discovery/file/FileBasedDiscoveryPluginTests.java

Lines changed: 0 additions & 47 deletions
This file was deleted.

0 commit comments

Comments
 (0)