Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

update to send api version 6 #10518

Closed

Conversation

jakelandis
Copy link
Contributor

This is part of effort to remove types from our internal indexes, Elasticsearch will be bumping the current monitoring API from 6 to 7. Only N-1 versions are supported, so this allows Logstash monitoring to continue to work on 7.0.

I did a quick spot of the code/payload/Kibana with version bump and all seemed well. However, another manual test would be appreciated.

This change needs to go back to 6.7.

Copy link
Contributor

@cachedout cachedout left a comment

Choose a reason for hiding this comment

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

👍 from Stack Monitoring

Copy link
Member

@jsvd jsvd left a comment

Choose a reason for hiding this comment

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

tested manually, LGTM

@elasticsearch-bot
Copy link

João Duarte merged this into the following branches!

Branch Commits
master ad65518
7.x 439ef7f
7.0 7db4fab
6.7 2930bd8

elasticsearch-bot pushed a commit that referenced this pull request Mar 7, 2019
elasticsearch-bot pushed a commit that referenced this pull request Mar 7, 2019
elasticsearch-bot pushed a commit that referenced this pull request Mar 7, 2019
@jsvd jsvd removed the v8.0.0 label Oct 31, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants