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

[elasticsearch] remove initial_master_nodes test#740

Merged
jmlrt merged 1 commit intoelastic:6.8from
jmlrt:fix-670-for-6x
Jul 23, 2020
Merged

[elasticsearch] remove initial_master_nodes test#740
jmlrt merged 1 commit intoelastic:6.8from
jmlrt:fix-670-for-6x

Conversation

@jmlrt
Copy link
Copy Markdown
Member

@jmlrt jmlrt commented Jul 20, 2020

test_initial_master_nodes_when_using_full_name_override has been added in af288a9 but is not valid with 6.x version.

This commit remove from 6.8 branch the test and elasticsearch.endpoints template which are only used with 7.x version.

Fixes elastic+helm-charts+6.8+template-testing job.

test_initial_master_nodes_when_using_full_name_override has been added in af288a9 but is not valid with 6.x version.

This commit remove from 6.8 branch the test and elasticsearch.endpoints template which are only used with 7.x version.
@jmlrt jmlrt added bug Something isn't working v6.8.11 labels Jul 20, 2020
@jmlrt jmlrt requested a review from a team July 20, 2020 09:41
Copy link
Copy Markdown
Contributor

@Conky5 Conky5 left a comment

Choose a reason for hiding this comment

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

LGTM

@jmlrt jmlrt merged commit 505e78f into elastic:6.8 Jul 23, 2020
@jmlrt jmlrt deleted the fix-670-for-6x branch July 23, 2020 06:15
@jmlrt jmlrt mentioned this pull request Oct 28, 2020
This was referenced Nov 17, 2020
@jmlrt jmlrt mentioned this pull request Dec 14, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

bug Something isn't working v6.8.11

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants