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

OPGOPS-1734 Enable ssl for elasticsearch5 #127

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

Conversation

carinadigital
Copy link
Contributor

No description provided.

@carinadigital carinadigital changed the title WIP-OPGOPS-1734 Enable ssl for elasticsearch5 WIP OPGOPS-1734 Enable ssl for elasticsearch5 Feb 16, 2017
@carinadigital carinadigital force-pushed the OPGOPS-1734 branch 2 times, most recently from dde8d57 to 2c19c05 Compare February 16, 2017 16:29
@carinadigital
Copy link
Contributor Author

Default for ES5 is SSL enabled. Just wondering if this is correct for integrating further upstream.

{{ else }}
xpack.security.transport.ssl.enabled: false
xpack.security.http.ssl.enabled: false
{{end}}
Copy link
Contributor

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

Copy link
Contributor Author

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.

Copy link
Contributor Author

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
Copy link
Contributor Author

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.

Copy link
Contributor

Choose a reason for hiding this comment

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

Overriding keeps it simple.

@carinadigital carinadigital changed the title WIP OPGOPS-1734 Enable ssl for elasticsearch5 OPGOPS-1734 Enable ssl for elasticsearch5 Feb 17, 2017
@@ -76,3 +77,5 @@ RUN ln -s /etc/sv/elasticsearch /etc/service/
RUN ln -sf /dev/stdout /var/log/elastic-scripts.log
Copy link
Contributor

Choose a reason for hiding this comment

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

Is this still relevant.

Copy link
Contributor Author

@carinadigital carinadigital Feb 17, 2017

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
Copy link
Contributor

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

@damacus
Copy link
Contributor

damacus commented Apr 3, 2018

Is this still needed?

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.

3 participants