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 monitoring HTTP end point #10528

Closed

Conversation

jakelandis
Copy link
Contributor

This commit changes /_xpack/monitoring/_bulk to /_monitoring/bulk.
The former is deprecated as of 7.0.0.

Relates elastic/elasticsearch#36130
Relates elastic/elasticsearch#35958


This prevents alot of deprecation warnings in the logs.

@jakelandis
Copy link
Contributor Author

jenkins test this please

@jsvd
Copy link
Member

jsvd commented Mar 11, 2019

jenkins test this please

This commit changes /_xpack/monitoring/_bulk to /_monitoring/bulk.
The former is deprecrated as 7.0.0.

Relates elastic/elasticsearch#36130
Relates elastic/elasticsearch#35958
@jakelandis jakelandis force-pushed the updated_monitoring_path branch from 7647b38 to 78dae9f Compare March 11, 2019 20:48
@jakelandis
Copy link
Contributor Author

Test failures were due #10533. This PR has been rebased with that fix.

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.

LGTM tested manually as well

@elasticsearch-bot
Copy link

João Duarte merged this into the following branches!

Branch Commits
master 4867674
7.x 3c3e769
7.0 dcde7e3

elasticsearch-bot pushed a commit that referenced this pull request Mar 12, 2019
This commit changes /_xpack/monitoring/_bulk to /_monitoring/bulk.
The former is deprecrated as 7.0.0.

Relates elastic/elasticsearch#36130
Relates elastic/elasticsearch#35958

Fixes #10528
elasticsearch-bot pushed a commit that referenced this pull request Mar 12, 2019
This commit changes /_xpack/monitoring/_bulk to /_monitoring/bulk.
The former is deprecrated as 7.0.0.

Relates elastic/elasticsearch#36130
Relates elastic/elasticsearch#35958

Fixes #10528
@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.

3 participants