Skip to content
This repository has been archived by the owner on Jun 24, 2022. It is now read-only.

[7.x] add support for elasticsearch 7.x #558

Merged
merged 27 commits into from
May 9, 2019
Merged

[7.x] add support for elasticsearch 7.x #558

merged 27 commits into from
May 9, 2019

Conversation

jmlrt
Copy link
Member

@jmlrt jmlrt commented May 2, 2019

Add support for Elasticsearch 7.x.

This should fix #554

@jmlrt jmlrt requested a review from Crazybus May 2, 2019 13:31
@jmlrt jmlrt self-assigned this May 2, 2019
@jmlrt
Copy link
Member Author

jmlrt commented May 6, 2019

jenkins test this please

@jmlrt
Copy link
Member Author

jmlrt commented May 6, 2019

jenkins test this please

Copy link
Contributor

@Crazybus Crazybus left a comment

Choose a reason for hiding this comment

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

Couple small changes but the rest LGTM!

tasks/elasticsearch-parameters.yml Outdated Show resolved Hide resolved
tasks/elasticsearch-parameters.yml Outdated Show resolved Hide resolved
tasks/main.yml Outdated Show resolved Hide resolved
test/integration/multi.yml Outdated Show resolved Hide resolved
xpack.security.enabled: True
xpack.security.authc.realms.file.file1.order: 0
xpack.security.authc.realms.native.native1.order: 1
es_config: "{{ es_config_7x if es_major_version == '7.x' else es_config_6x }}"
Copy link
Contributor

Choose a reason for hiding this comment

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

I like this trick a lot :)

tasks/main.yml Outdated Show resolved Hide resolved
jmlrt and others added 4 commits May 7, 2019 17:11
- add different tests for transport port for 6.x and 7.x
- fix nodes discovery for 6.x and 7.x
…eption" error when creating templates in the same play
@jmlrt jmlrt requested a review from Crazybus May 7, 2019 17:52
@jmlrt jmlrt merged commit a1c8188 into master May 9, 2019
@jmlrt jmlrt deleted the es-7.x branch May 9, 2019 07:06
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.

[Feature] 7.0 support
2 participants