Skip to content
This repository has been archived by the owner on Apr 12, 2019. It is now read-only.

[feature request] file based discovery #23

Closed
hanabi1224 opened this issue Mar 29, 2017 · 7 comments
Closed

[feature request] file based discovery #23

hanabi1224 opened this issue Mar 29, 2017 · 7 comments

Comments

@hanabi1224
Copy link

With file based discovery, i think it will make it easier to upgrade to latest ES by getting rid of azure-runtime java plugin
https://www.elastic.co/guide/en/elasticsearch/plugins/current/discovery-file.html

@garvincasimir
Copy link
Owner

Getting rid of the java runtime plugin in favor of this would definitely simplify the upgrade process for me (Every version has breaking changes). However, I am not sure how it would help someone consuming this library. For example, you would be required to include discovery-file-5.3.0.zip in either your visual studio project or your Installs storage container for ES 5.3.0. At least the java plugin is included in the Nuget package.

@hanabi1224
Copy link
Author

hanabi1224 commented Mar 31, 2017

@garvincasimir I think you can have a config field of all required plugin names, and install plugins in ElasticSearchManager with elasticsearch-plugin.bat, and discovery-file can be default option

@garvincasimir
Copy link
Owner

garvincasimir commented Mar 31, 2017

I can add the option. To be honest, I purposely avoided relying on the built in plugin installer because i don't believe a running system should rely on Github and similar sites for packages once deployed.

@garvincasimir
Copy link
Owner

Decided to remove the java plugin and use Zen Discovery. Discovery-file not required. Upgrades should be a lot easier from this point.

@hanabi1224
Copy link
Author

@garvincasimir Does it work when you scale up the number of instances?

@garvincasimir
Copy link
Owner

Yes it does work when you scale up the number of instances. Please let me know if you encounter any problems.

I also added the option for installing plugins with a pipe separated config value. Note that it is significantly slower than the 2 other available options for installing plugins.

@hanabi1224
Copy link
Author

@garvincasimir awesome, thanks!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants