Skip to content

[7.x] Monitoring telemetry collection interval (#34609)#35692

Merged
Bamieh merged 1 commit into7.xfrom
backport/7.x/pr-34609
Apr 27, 2019
Merged

[7.x] Monitoring telemetry collection interval (#34609)#35692
Bamieh merged 1 commit into7.xfrom
backport/7.x/pr-34609

Conversation

@Bamieh
Copy link
Copy Markdown
Contributor

@Bamieh Bamieh commented Apr 27, 2019

Backports the following commits to 7.x:

* Only collect usage data once a day from kibana monitoring

* isUsageCollector

* bulk uploader tests

* linting

* condition filter

* checkout yarn.lock from master

* update mappings for functional tests

* debugging for refactoring

* support legacy mappings

* self code review

* isUsageCollector

* fix mock

* live coding session with pickypg

* self code review

* Update x-pack/plugins/xpack_main/server/lib/telemetry/monitoring/get_high_level_stats.js

Co-Authored-By: Bamieh <ahmadbamieh@gmail.com>

* update mappings
@Bamieh Bamieh added the backport This PR is a backport of another PR label Apr 27, 2019
@Bamieh Bamieh requested review from a team as code owners April 27, 2019 01:15
@Bamieh Bamieh requested a review from a team April 27, 2019 01:15
@Bamieh Bamieh changed the base branch from master to 7.x April 27, 2019 01:16
Copy link
Copy Markdown
Contributor

@peterschretlen peterschretlen left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Copy Markdown
Member

@pickypg pickypg left a comment

Choose a reason for hiding this comment

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

LGTM

@elasticmachine
Copy link
Copy Markdown
Contributor

💔 Build Failed

@pickypg
Copy link
Copy Markdown
Member

pickypg commented Apr 27, 2019

Unrelated security failures: jenkins please test this

@elasticmachine
Copy link
Copy Markdown
Contributor

💔 Build Failed

@pickypg
Copy link
Copy Markdown
Member

pickypg commented Apr 27, 2019

The test failures are consistent, albeit unrelated to this PR.

#35692 has the same issue and the 7.x tests are failing for the same reason.

@peterschretlen
Copy link
Copy Markdown
Contributor

7.x branch is clear again. retest

@peterschretlen
Copy link
Copy Markdown
Contributor

retest

@elasticmachine
Copy link
Copy Markdown
Contributor

💔 Build Failed

@ycombinator
Copy link
Copy Markdown
Contributor

retest

@elasticmachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

@Bamieh Bamieh merged commit ad6676b into 7.x Apr 27, 2019
@pickypg pickypg deleted the backport/7.x/pr-34609 branch April 28, 2019 22:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport This PR is a backport of another PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants