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

Cherry-pick #8415 to 6.x: Heartbeat autodiscover #8593

Merged
merged 1 commit into from
Oct 16, 2018

Conversation

andrewvc
Copy link
Contributor

@andrewvc andrewvc commented Oct 5, 2018

Cherry-pick of PR #8415 to 6.x branch. Original message:

This is a complete PR for heartbeat automatic reload.

@@ -19,6 +19,8 @@ https://github.com/elastic/beats/compare/v6.4.0...6.x[Check the HEAD diff]

*Heartbeat*

- Added autodiscovery support {pull}8415[8415]
Copy link
Contributor

Choose a reason for hiding this comment

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

Why is this a breaking change?

Copy link
Contributor

@ruflin ruflin left a comment

Choose a reason for hiding this comment

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

Overall LGTM expect changelog.

@andrewvc
Copy link
Contributor Author

andrewvc commented Oct 13, 2018 via email

@andrewvc
Copy link
Contributor Author

@ruflin updated the changelog entry and re-pushed.

Copy link
Contributor

@ruflin ruflin left a comment

Choose a reason for hiding this comment

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

LGTM. Will need rebase on top of #8619 as soon as it's backported.

Autodiscover support for Heartbeat + tests

(cherry picked from commit cff3e40)
@andrewvc andrewvc merged commit 09e26ad into elastic:6.x Oct 16, 2018
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.

2 participants