Skip to content

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

Merged
sayden merged 4 commits intoelastic:masterfrom
sayden:migration/mb/reportv2/etcd/self
Feb 22, 2019
Merged

[Metricbeat] Migrate ETCD self to use ReporterV2 interface#10816
sayden merged 4 commits intoelastic:masterfrom
sayden:migration/mb/reportv2/etcd/self

Conversation

@sayden
Copy link
Copy Markdown
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
Copy Markdown
Contributor Author

sayden commented Feb 19, 2019

jenkins, test this please

@sayden
Copy link
Copy Markdown
Contributor Author

sayden commented Feb 19, 2019

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

@sayden
Copy link
Copy Markdown
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
Copy Markdown
Contributor Author

sayden commented Feb 20, 2019

"leaderinfo": {
"leader": "8e9e05c52164694d",
"starttime": "2017-12-07T07:20:33.241031712Z",
"uptime": "9m4.919923089s"
Copy link
Copy Markdown
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
Copy Markdown
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