Skip to content
This repository was archived by the owner on Mar 1, 2019. It is now read-only.

update to elastic search 5.5.0 #4

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

yanns
Copy link

@yanns yanns commented Jul 28, 2017

No description provided.

@yanns
Copy link
Author

yanns commented Jul 28, 2017

Fix #3

Dockerfile Outdated

RUN bin/plugin install lmenezes/elasticsearch-kopf/v2.1.2
RUN bin/plugin install io.fabric8/elasticsearch-cloud-kubernetes/2.4.0_01
RUN bin/elasticsearch-plugin install io.fabric8:elasticsearch-cloud-kubernetes:5.5.0
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can you upgrade to 5.5.1, now fabric8:elasticsearch-cloud-kubernetes plugin is on 5.5.1. see here

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Now 5.5.2 is out

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

path:
plugins: /usr/share/elasticsearch/plugins
zen:
minimum_master_nodes: ${NUMBER_OF_MASTERS}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we call that MINIMUM_MASTER_NODES ?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@kledk
Copy link

kledk commented Sep 5, 2017

Please merge!

@simonswine
Copy link
Contributor

Some minor things to address, see my comments. Maybe you want to have a look?: @dippynark

@yanns
Copy link
Author

yanns commented Sep 6, 2017

I'll try to take time today to address the comments.
Thx all for the feedback!

@ypougeol
Copy link

ypougeol commented Oct 9, 2017

Hello guys, any release soon? You're doing a good job

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

Successfully merging this pull request may close these issues.

5 participants