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

Send system_api_version=7 from beats 7.0.0 onwards #11197

Merged
merged 1 commit into from
Mar 13, 2019

Conversation

ycombinator
Copy link
Contributor

@ycombinator ycombinator commented Mar 11, 2019

Resolves #11192.

Starting Elasticsearch 7.0.0, the bulk monitoring API is expecting the system_api_version parameter to be set to 7 (used to be 6 before). This PR makes that change for all Beats.

@ycombinator ycombinator requested review from urso, jakelandis and a team March 11, 2019 19:56
@elasticmachine
Copy link
Collaborator

Pinging @elastic/stack-monitoring

@ycombinator ycombinator requested a review from a team as a code owner March 11, 2019 19:56
Copy link

@jakelandis jakelandis left a comment

Choose a reason for hiding this comment

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

LGTM

@ycombinator
Copy link
Contributor Author

Blocked by #11204.

@ycombinator ycombinator added the in progress Pull request is currently in progress. label Mar 11, 2019
@ycombinator ycombinator added review and removed blocked in progress Pull request is currently in progress. labels Mar 13, 2019
@ycombinator ycombinator merged commit 8169159 into elastic:master Mar 13, 2019
@ycombinator ycombinator removed the needs_backport PR is waiting to be backported to other branches. label Mar 13, 2019
ycombinator added a commit that referenced this pull request Mar 14, 2019
@ycombinator ycombinator added the needs_backport PR is waiting to be backported to other branches. label Apr 3, 2019
@ycombinator ycombinator deleted the mon-sys-api-v7 branch December 25, 2019 11:17
@ycombinator ycombinator removed the needs_backport PR is waiting to be backported to other branches. label Jan 15, 2020
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.

[monitoring] Bump system_api_version to 7
4 participants