Skip to content
This repository was archived by the owner on Jan 31, 2024. It is now read-only.

Remove component settings from AbstractComponent #50

Closed
dadoonet opened this issue Apr 24, 2015 · 4 comments
Closed

Remove component settings from AbstractComponent #50

dadoonet opened this issue Apr 24, 2015 · 4 comments
Assignees
@dadoonet
Labels
2.5.1 2.6.0 update
Milestone
2.6.0

Comments

@dadoonet
Copy link
Member

Related to elastic/elasticsearch#9919
@dadoonet dadoonet added this to the 2.5.1 milestone Apr 24, 2015
@dadoonet dadoonet self-assigned this Apr 24, 2015
dadoonet added a commit that referenced this issue Apr 24, 2015
@dadoonet
Remove component settings from AbstractComponent
bc5b642 
Related to elastic/elasticsearch#9919

Closes #50.
(cherry picked from commit c70e32d)
dadoonet added a commit that referenced this issue Apr 24, 2015
@dadoonet
Remove component settings from AbstractComponent
26b7cf1 
Related to elastic/elasticsearch#9919

Closes #50.
(cherry picked from commit c70e32d)
@dadoonet dadoonet added the 2.6.0 label Jun 10, 2015
@schonfeld
Copy link

hey @dadoonet ... pretty sure removing all that logic from GceDiscovery.java broke this plugin's ability to discover other elasticsearch nodes on GCE... see issues #53 & #54.

@dianat-porch
Copy link

Having problems with ES 1.7.1 and plugin 2.7.0 as well. Nodes aren't discovering each other. @schonfeld could it be related?

@schonfeld
Copy link

@dianat-porch most likely, this is the cause. We're using v2.5.0 w/ ES 1.7. Works just fine... For now :)

@hagait
Copy link

hagait commented Sep 10, 2015

Thanks @schonfeld, It seems to work fine with v2.5.0 and ES 1.7.1.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@dadoonet dadoonet

Labels
2.5.1 2.6.0 update
Projects
None yet
Milestone
2.6.0
Development

No branches or pull requests
4 participants
@schonfeld @dadoonet @dianat-porch @hagait