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.

Sign up for GitHub

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

Jump to bottom

Add formatting to pressure fields in system/process #27374

Merged
merged 3 commits into from
Aug 16, 2021
Merged

Add formatting to pressure fields in system/process #27374

merged 3 commits into from
Aug 16, 2021

Conversation

fearful-symmetry
Copy link
Contributor

@fearful-symmetry fearful-symmetry commented Aug 13, 2021 •
edited
Loading

What does this PR do?

I forgot to add field formatting directives while adding cgroupsV2 support. This adds them to *.pressure.*.pct.

Why is it important?

The formatting directives should be there.

Checklist

  • My code follows the style guidelines of this project
  • I have added an entry in CHANGELOG.next.asciidoc or CHANGELOG-developer.next.asciidoc.

@fearful-symmetry fearful-symmetry added Team:Integrations Label for the Integrations team v7.15.0 labels Aug 13, 2021
@fearful-symmetry fearful-symmetry requested a review from a team August 13, 2021 17:40
@fearful-symmetry fearful-symmetry self-assigned this Aug 13, 2021
@botelastic botelastic bot added the needs_team Indicates that the issue/PR needs a Team:* label label Aug 13, 2021
@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 Aug 13, 2021
@elasticmachine
Copy link
Collaborator

elasticmachine commented Aug 13, 2021 •
edited by jenkins-beats-ci bot
Loading

💚 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 preview

Expand to view the summary

Build stats

  • Start Time: 2021-08-13T17:54:22.568+0000

  • Duration: 111 min 23 sec

  • Commit: 20f2577

Test stats 🧪

Test Results
Failed 0
Passed 9127
Skipped 2426
Total 11553

Trends 🧪

Image of Build Times

Image of Tests

💚 Flaky test report

Tests succeeded.

Expand to view the summary

Test stats 🧪

Test Results
Failed 0
Passed 9127
Skipped 2426
Total 11553

kaiyan-sheng
kaiyan-sheng approved these changes Aug 16, 2021
View reviewed changes
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 afde058 into elastic:master Aug 16, 2021
@fearful-symmetry fearful-symmetry mentioned this pull request Aug 17, 2021
Merged
2 tasks
fearful-symmetry added a commit to fearful-symmetry/beats that referenced this pull request Aug 17, 2021
@fearful-symmetry
Add formatting to pressure fields in system/process (elastic#27374)
d1a76cd 
* add formatting to pressure fields

* add changelog

(cherry picked from commit afde058)
fearful-symmetry added a commit that referenced this pull request Aug 17, 2021
@fearful-symmetry
Add formatting to pressure fields in system/process (#27374) (#27430)
7cc33f5 
* add formatting to pressure fields

* add changelog

(cherry picked from commit afde058)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kaiyan-sheng kaiyan-sheng kaiyan-sheng approved these changes

Assignees

@fearful-symmetry fearful-symmetry

Labels
Team:Integrations Label for the Integrations team v7.15.0
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@fearful-symmetry @elasticmachine @kaiyan-sheng