Skip to content

Update benchmark metrics memory reader probe interval#2228

Merged
ambushwork merged 1 commit into
developfrom
yl/metrics/fix-memory-probe
Sep 3, 2024
Merged

Update benchmark metrics memory reader probe interval#2228
ambushwork merged 1 commit into
developfrom
yl/metrics/fix-memory-probe

Conversation

@ambushwork

Copy link
Copy Markdown
Member

What does this PR do?

Update benchmark metrics memory reader probe interval

Logic changes:

  • Before: Memory reader reads the stats file directly for the memory data when readVitalData() is called every 10 seconds.
  • After: Memory reader keeps reading the stats file every 500ms internally and keeps the max memory data until readVitalData is called to output this result.

Review checklist (to be filled by reviewers)

  • Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
  • Make sure you discussed the feature or bugfix with the maintaining team in an Issue
  • Make sure each commit and the PR mention the Issue number (cf the CONTRIBUTING doc)

@ambushwork
ambushwork requested review from a team as code owners September 2, 2024 10:05
@ambushwork
ambushwork force-pushed the yl/metrics/fix-memory-probe branch 3 times, most recently from c7ad39a to 7bb64f2 Compare September 2, 2024 14:53
@ambushwork
ambushwork force-pushed the yl/metrics/fix-memory-probe branch from 7bb64f2 to 88e0b66 Compare September 3, 2024 09:59
@codecov-commenter

codecov-commenter commented Sep 3, 2024

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 69.97%. Comparing base (87ac85e) to head (88e0b66).
Report is 2 commits behind head on develop.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #2228      +/-   ##
===========================================
- Coverage    70.12%   69.97%   -0.16%     
===========================================
  Files          727      727              
  Lines        27052    27013      -39     
  Branches      4558     4551       -7     
===========================================
- Hits         18970    18900      -70     
- Misses        6818     6829      +11     
- Partials      1264     1284      +20     

see 29 files with indirect coverage changes

@mariusc83 mariusc83 left a comment

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.

Tks for the updates !!

@ambushwork
ambushwork requested a review from 0xnm September 3, 2024 12:22
@ambushwork
ambushwork merged commit fc30720 into develop Sep 3, 2024
@ambushwork
ambushwork deleted the yl/metrics/fix-memory-probe branch September 3, 2024 14:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants