Skip to content
This repository has been archived by the owner on May 16, 2023. It is now read-only.

[logstash] using https to connect to elasticsearch #1623

Merged
merged 40 commits into from
Mar 21, 2022

Conversation

framsouza
Copy link
Contributor

This PR will use https to connect into elasticsearch by default

@framsouza framsouza requested a review from jmlrt March 17, 2022 16:15
Copy link
Member

@jmlrt jmlrt left a comment

Choose a reason for hiding this comment

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

This PR should be rebased after #1624 merge to fix the Certificate for <elasticsearch-master> doesn't match any of the subject alternative names errors.

logstash/templates/statefulset.yaml Outdated Show resolved Hide resolved
logstash/values.yaml Outdated Show resolved Hide resolved
logstash/values.yaml Outdated Show resolved Hide resolved
logstash/values.yaml Outdated Show resolved Hide resolved
jmlrt added a commit to jmlrt/helm-charts that referenced this pull request Mar 18, 2022
This also seem to be required to Logstash can reuse the Elasticsearch
certificates to connect to Elasticsearch Service.

Follow-up of elastic#1624
Required for elastic#1623
@framsouza framsouza self-assigned this Mar 21, 2022
jmlrt added a commit that referenced this pull request Mar 21, 2022
This also seem to be required to Logstash can reuse the Elasticsearch
certificates to connect to Elasticsearch Service.

Follow-up of #1624
Required for #1623
@framsouza
Copy link
Contributor Author

jenkins test this please

Copy link
Member

@jmlrt jmlrt left a comment

Choose a reason for hiding this comment

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

  • Some spaces added by mistakes to remove
  • Let's verify that the next CI test is OK for logstash

The rest LGTM.

logstash/values.yaml Outdated Show resolved Hide resolved
logstash/values.yaml Outdated Show resolved Hide resolved
@jmlrt jmlrt merged commit fb00805 into elastic:main Mar 21, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants