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

[Metricbeat] Migrate ETCD self to use ReporterV2 interface #10816

Merged
merged 4 commits into from
Feb 22, 2019

Conversation

sayden
Copy link
Contributor

@sayden sayden commented Feb 18, 2019

Refer to #10774 for more info

@sayden sayden added Metricbeat Metricbeat Team:Integrations Label for the Integrations team labels Feb 18, 2019
@sayden sayden self-assigned this Feb 18, 2019
@sayden sayden requested a review from a team as a code owner February 18, 2019 22:21
@sayden
Copy link
Contributor Author

sayden commented Feb 19, 2019

jenkins, test this please

@sayden
Copy link
Contributor Author

sayden commented Feb 19, 2019

jenkins, test this please https://travis-ci.org/elastic/beats/jobs/495522479#L172

@sayden
Copy link
Contributor Author

sayden commented Feb 19, 2019

jenkins, test this again please because No output has been received in the last 10m0s, this potentially indicates a stalled build or something wrong with the build itself.

@sayden
Copy link
Contributor Author

sayden commented Feb 20, 2019

@@ -9,8 +9,8 @@
"id": "8e9e05c52164694d",
"leaderinfo": {
"leader": "8e9e05c52164694d",
"starttime": "2017-12-07T07:20:33.241031712Z",
"uptime": "9m4.919923089s"
Copy link
Member

Choose a reason for hiding this comment

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

For another PR, we could convert this to seconds.

Copy link
Contributor Author

@sayden sayden Feb 22, 2019

Choose a reason for hiding this comment

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

Can you open an issue to enhance this, please? 😉

@sayden sayden merged commit 86e77e2 into elastic:master Feb 22, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Metricbeat Metricbeat Team:Integrations Label for the Integrations team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants