Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

Refactor Resource Monitoring#6554

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Conversation

@evgenyfedorov2
Copy link
Member

@evgenyfedorov2evgenyfedorov2 commentedJun 30, 2025•
edited
Loading

  1. RenameCalculateCpuUsageWithoutHostDelta toUseLinuxCpuUsageV2.
  2. RemoveUseDeltaNrPeriodsForCpuCalculation option and merge its logic withUseLinuxCpuUsageV2.
  3. Remove debug logs and metrics.
  4. Minor code massaging.
Microsoft Reviewers:Open in CodeFlow

@evgenyfedorov2evgenyfedorov2force-pushed theusers/evgenyfedorov2/refactor_linux_cgroup_options branch 2 times, most recently from07b5619 to54a0b83CompareJune 30, 2025 09:43
Update CompatibilitySuppressionsRemove unnecessary logging and metricsRename CalculateCpuUsageWithoutHostDelta to UseLinuxCpuUsageV2Remove UseDeltaNrPeriosForCpuCalculaton
@evgenyfedorov2evgenyfedorov2force-pushed theusers/evgenyfedorov2/refactor_linux_cgroup_options branch from54a0b83 to4e16faeCompareJune 30, 2025 10:09
@evgenyfedorov2evgenyfedorov2 marked this pull request as ready for reviewJune 30, 2025 11:04
@evgenyfedorov2evgenyfedorov2 requested review froma team ascode ownersJune 30, 2025 11:04
@evgenyfedorov2
Copy link
MemberAuthor

@nivebb8 do you want to review this? We are removing debug logs, metrics, and adjusting code to use only one boolean flag instead of two.

nivebb8 reacted with thumbs up emoji

evgenyfedorov2 added2 commitsJuly 3, 2025 07:04
…ptions# Conflicts:#src/Libraries/Microsoft.Extensions.Diagnostics.ResourceMonitoring/CompatibilitySuppressions.xml
@evgenyfedorov2evgenyfedorov2enabled auto-merge (squash)July 3, 2025 05:08
@evgenyfedorov2evgenyfedorov2 merged commit95a6013 intodotnet:mainJul 3, 2025
6 checks passed
@github-actionsgithub-actionsbot locked and limited conversation to collaboratorsAug 2, 2025
Sign up for freeto subscribe to this conversation on GitHub. Already have an account?Sign in.

Reviewers

@amadeuszlamadeuszlamadeuszl approved these changes

+1 more reviewer

@nivebb8nivebb8nivebb8 left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

@evgenyfedorov2evgenyfedorov2

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@evgenyfedorov2@amadeuszl@nivebb8
[8]ページ先頭
©2009-2025 Movatter.jp