-
Notifications
You must be signed in to change notification settings - Fork 3
OPGOPS-1734 Enable ssl for elasticsearch5 #127
base: master
Are you sure you want to change the base?
Conversation
dde8d57
to
2c19c05
Compare
Default for ES5 is SSL enabled. Just wondering if this is correct for integrating further upstream. |
2c19c05
to
ff66b57
Compare
{{ else }} | ||
xpack.security.transport.ssl.enabled: false | ||
xpack.security.http.ssl.enabled: false | ||
{{end}} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
how does elasticsearch behave if the certs are in place, but ssl is disabled
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As expected. elasticsearch starts but communication with external client and also between nodes is via HTTP.
The ports don't change when SSL is enabled.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We get default authorisation too. use
curl -k -u elastic:changeme https://localhost:9200
@@ -0,0 +1,33 @@ | |||
#!/bin/sh |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could we use a different init number here so we don't overwrite the base image behaviour?
They could survive in parallel quite happily.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Overriding keeps it simple.
d8473ef
to
876daaa
Compare
@@ -76,3 +77,5 @@ RUN ln -s /etc/sv/elasticsearch /etc/service/ | |||
RUN ln -sf /dev/stdout /var/log/elastic-scripts.log |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this still relevant.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't have the context of where elastic-scripts was used. They were empty during my runs
Shall I remove?
|
||
elasticsearch02: | ||
build: elasticsearch5 | ||
mem_limit: 1000000000 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this is known to cause issues. Use ES_JAVA_OPTS instead to limit the memory
Is this still needed? |
No description provided.