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

[Fleet] Always collect agent telemetry #93198

Merged
merged 1 commit into from
Mar 2, 2021

Conversation

skh
Copy link
Contributor

@skh skh commented Mar 2, 2021

Summary

Fixes #92814

This fixes a bug where agent telemetry was only being collected when fleet server was correctly set up. In the case where fleet server wasn't enabled at all, no agent telemetry was returned. This fix is actually only needed for 7.12, because in 7.13 the feature flag for fleet server will no longer be there.

This means this needs to be merged and backported to 7.12 and 7.x before #92422 is merged, and probably will cause conflicts in #92422

cc @nchaulet for awareness.

How to test this

  • enroll an agent
  • observe the agent is listed on the Fleet overview page and in Agents
  • look at the telemetry output either by calling http://localhost:5601/BASEPATH/api/stats?extended=true or by going to Stack Management -> Advanced Settings -> Usage Data (scroll all the way down) and clicking on "See examples of the cluster data we collect"
  • search for the last occurrence of fleet in the telemetry output and verify that you see the same numbers under agents as on the Fleet overview page in the UI

@skh skh requested a review from a team as a code owner March 2, 2021 11:52
@botelastic botelastic bot added the Team:Fleet Team label for Observability Data Collection Fleet team label Mar 2, 2021
@elasticmachine
Copy link
Contributor

Pinging @elastic/fleet (Team:Fleet)

@skh skh self-assigned this Mar 2, 2021
@skh skh added release_note:skip Skip the PR/issue when compiling release notes v7.12.0 v7.13.0 v8.0.0 Feature:Fleet Fleet team's agent central management project labels Mar 2, 2021
@skh skh requested a review from nchaulet March 2, 2021 12:02
@skh skh changed the title Collect agent telemetry even when fleet server is disabled. [Fleet] Always collect agent telemetry Mar 2, 2021
@kibanamachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

✅ unchanged

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

cc @skh

Copy link
Member

@nchaulet nchaulet left a comment

Choose a reason for hiding this comment

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

🚀

@skh skh merged commit ca25e51 into elastic:master Mar 2, 2021
skh added a commit to skh/kibana that referenced this pull request Mar 2, 2021
skh added a commit to skh/kibana that referenced this pull request Mar 2, 2021
gmmorris added a commit to gmmorris/kibana that referenced this pull request Mar 2, 2021
…bana into task-manager/docs-monitoring

* 'task-manager/docs-monitoring' of github.com:gmmorris/kibana:
  [ILM] Allow multiple searchable snapshot actions (elastic#92789)
  Improve consistency for display of management items (elastic#92694)
  skip flaky suite (elastic#93152)
  skip flaky suite (elastic#93152)
  [ILM] Refactor edit_policy client integration tests into separate feature files (elastic#92826)
  Add developer documentation about the building blocks we offer plugin developers (elastic#92743)
  [Security Solution] Case ui enhancement (elastic#91863)
  [Security Solution] [Detections] Updates warning message when no indices match provided index patterns (elastic#93094)
  Collect agent telemetry even when fleet server is disabled. (elastic#93198)
  [Lens] Fix runtime validation error message (elastic#93195)
  [Lens] Remove warning about ordinal x-domain (elastic#93049)
  [Security Solution] Fixes the Customize Event Renderers modal by removing the EuiOverlayMask (elastic#93150)
  Cleanup Security plugin imports (elastic#93056)
  [Security Solution] - Bug fixes (elastic#92294)
  Updated doc links (elastic#92968)
  [ML] Transforms: Fixes chart histograms for runtime fields. (elastic#93028)
  [chore] Enable core's eslint rule: `@ts-expect-error` (elastic#93086)
gmmorris added a commit to gmmorris/kibana that referenced this pull request Mar 2, 2021
* master: (199 commits)
  Convert Canvas docs to MDX for use in Elastic Docs (elastic#91969)
  [Bazel] More resilient Workspace Status (elastic#93244)
  [Discover] Change icon of saved search in open search panel and embeddable selection (elastic#93001)
  [Workplace Search] Role Mappings to Kibana (elastic#93123)
  [Fleet] Use type-only imports where possible (elastic#92979)
  [Lens] Set pie chart slices sorted clockwise (elastic#92617)
  Remove ms label from CPU load on status page (elastic#92836)
  [App Search] Migrate Create Meta Engine View (elastic#92127)
  [Time to Visualize] Disable Visualize URL Tracker When Linked to OriginatingApp (elastic#92917)
  [ILM] Allow multiple searchable snapshot actions (elastic#92789)
  Improve consistency for display of management items (elastic#92694)
  skip flaky suite (elastic#93152)
  skip flaky suite (elastic#93152)
  [ILM] Refactor edit_policy client integration tests into separate feature files (elastic#92826)
  Add developer documentation about the building blocks we offer plugin developers (elastic#92743)
  [Security Solution] Case ui enhancement (elastic#91863)
  [Security Solution] [Detections] Updates warning message when no indices match provided index patterns (elastic#93094)
  Collect agent telemetry even when fleet server is disabled. (elastic#93198)
  [Lens] Fix runtime validation error message (elastic#93195)
  [Lens] Remove warning about ordinal x-domain (elastic#93049)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature:Fleet Fleet team's agent central management project release_note:skip Skip the PR/issue when compiling release notes Team:Fleet Team label for Observability Data Collection Fleet team v7.12.0 v7.13.0 v8.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Incorrect Agents count under telemetry collection.
4 participants