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] Remove io.time from windows #22237

Merged
merged 4 commits into from
Oct 30, 2020

Conversation

fearful-symmetry
Copy link
Contributor

What does this PR do?

It looks like diskio.io.time is a unix-only field that reports an incorrect zero. We accidentally marked it as "no incorrect zero" on our massive spreadsheet of metrics inventory, so it was overlooked until now.

Why is it important?

We don't want to report "incorrect zero" metrics.

Checklist

  • My code follows the style guidelines of this project
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • I have added tests that prove my fix is effective or that my feature works
  • I have added an entry in CHANGELOG.next.asciidoc or CHANGELOG-developer.next.asciidoc.

How to test this PR locally

  • Pull and build on a Windows machine
  • Run the diskio metricset and insure that diskio.io.time is not reported.
  • Pull and build on linux, insure that diskio.io.time is reported.

Related issues

@fearful-symmetry fearful-symmetry added bug needs_backport PR is waiting to be backported to other branches. labels Oct 28, 2020
@fearful-symmetry fearful-symmetry requested a review from a team October 28, 2020 16:05
@fearful-symmetry fearful-symmetry self-assigned this Oct 28, 2020
@botelastic botelastic bot added the needs_team Indicates that the issue/PR needs a Team:* label label Oct 28, 2020
@fearful-symmetry fearful-symmetry added the Team:Integrations Label for the Integrations team label Oct 28, 2020
@elasticmachine
Copy link
Collaborator

Pinging @elastic/integrations (Team:Integrations)

@botelastic botelastic bot removed the needs_team Indicates that the issue/PR needs a Team:* label label Oct 28, 2020
@elasticmachine
Copy link
Collaborator

elasticmachine commented Oct 28, 2020

💚 Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS
Pipeline View Test View Changes Artifacts preview

Expand to view the summary

Build stats

  • Build Cause: [Pull request #22237 updated]

  • Start Time: 2020-10-29T22:24:05.718+0000

  • Duration: 67 min 18 sec

Test stats 🧪

Test Results
Failed 0
Passed 2247
Skipped 508
Total 2755

@urso
Copy link

urso commented Oct 29, 2020

The test failure looks related to the change.

@fearful-symmetry
Copy link
Contributor Author

Yep, didn't notice it yesterday. Should be fixed.

Copy link
Contributor

@kaiyan-sheng kaiyan-sheng left a comment

Choose a reason for hiding this comment

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

LGTM

@fearful-symmetry fearful-symmetry merged commit 77882fc into elastic:master Oct 30, 2020
v1v added a commit to v1v/beats that referenced this pull request Oct 30, 2020
* upstream/master:
  adding timeout options to cisco umbrella and fixing the missing config options in module config.yml (elastic#22295)
  [Metricbeat] Remove io.time from windows (elastic#22237)
  remove debug messages (elastic#22282)
  dependency upgrade: ecszap v0.3.0 (elastic#22267)
  [repo] Add docs + cleanup labels to the categories in the incoming PR template (elastic#22280)
v1v added a commit to v1v/beats that referenced this pull request Oct 30, 2020
* upstream/master:
  adding timeout options to cisco umbrella and fixing the missing config options in module config.yml (elastic#22295)
  [Metricbeat] Remove io.time from windows (elastic#22237)
  remove debug messages (elastic#22282)
  dependency upgrade: ecszap v0.3.0 (elastic#22267)
  [repo] Add docs + cleanup labels to the categories in the incoming PR template (elastic#22280)
fearful-symmetry added a commit to fearful-symmetry/beats that referenced this pull request Oct 30, 2020
* remove io.time from windows

* fix tests

* add changelog entry

(cherry picked from commit 77882fc)
@fearful-symmetry fearful-symmetry added v7.11.0 and removed needs_backport PR is waiting to be backported to other branches. labels Oct 30, 2020
fearful-symmetry added a commit to fearful-symmetry/beats that referenced this pull request Oct 30, 2020
* remove io.time from windows

* fix tests

* add changelog entry

(cherry picked from commit 77882fc)
fearful-symmetry added a commit that referenced this pull request Oct 30, 2020
…22314)

* [Metricbeat] Remove io.time from windows (#22237)

* remove io.time from windows

* fix tests

* add changelog entry

(cherry picked from commit 77882fc)

* fix changelog
fearful-symmetry added a commit that referenced this pull request Oct 30, 2020
…22313)

* [Metricbeat] Remove io.time from windows (#22237)

* remove io.time from windows

* fix tests

* add changelog entry

(cherry picked from commit 77882fc)

* fix changelog
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Team:Integrations Label for the Integrations team v7.10.0 v7.11.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants