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

Fix Process Metric calculation in CgroupsV2#6321

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

Merged
evgenyfedorov2 merged 5 commits intodotnet:mainfromnivebb8:nive/processMetricBugFix
Apr 22, 2025

Conversation

@nivebb8
Copy link
Contributor

@nivebb8nivebb8 commentedApr 21, 2025
edited by dotnet-policy-servicebot
Loading

Missed updating the process metric in first check-in. Updating the same to use correct calculations behind the flag, and to ensure CpuUtilization() is not invoked at all.

Microsoft Reviewers:Open in CodeFlow

Copy link
Member

@evgenyfedorov2evgenyfedorov2 left a comment

Choose a reason for hiding this comment

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

Why do we need this change?
TheResourceUtilizationInstruments.ProcessCpuUtilization metric is deprecated (unofficially).ResourceUtilizationInstruments.ContainerCpuRequestUtilization is the replacement and it already has the right calculation logic

@dotnet-policy-servicedotnet-policy-servicebot added the waiting-author-feedback 📭The author of this issue needs to respond in order for us to continue investigating this issue. labelApr 22, 2025
@dotnet-policy-servicedotnet-policy-servicebot removed the waiting-author-feedback 📭The author of this issue needs to respond in order for us to continue investigating this issue. labelApr 22, 2025
@nivebb8nivebb8 requested a review froma team as acode ownerApril 22, 2025 13:57
@evgenyfedorov2evgenyfedorov2enabled auto-merge (squash)April 22, 2025 15:37
@evgenyfedorov2evgenyfedorov2 merged commite5b6b02 intodotnet:mainApr 22, 2025
6 checks passed
@evgenyfedorov2
Copy link
Member

/backport to release/9.4

@github-actions
Copy link
Contributor

Started backporting torelease/9.4:https://github.com/dotnet/extensions/actions/runs/14599507495

@github-actions
Copy link
Contributor

@evgenyfedorov2 an error occurred while backporting to "release/9.4", please check the run log for details!

Error:@evgenyfedorov2 is not a repo collaborator, backporting is not allowed. If you're a collaborator please make sure your dotnet team membership visibility is set to Public onhttps://github.com/orgs/dotnet/people?query=evgenyfedorov2

@evgenyfedorov2
Copy link
Member

/backport to release/9.4

@github-actions
Copy link
Contributor

Started backporting torelease/9.4:https://github.com/dotnet/extensions/actions/runs/14599549296

@github-actionsgithub-actionsbot locked and limited conversation to collaboratorsMay 23, 2025
Sign up for freeto subscribe to this conversation on GitHub. Already have an account?Sign in.

Reviewers

@evgenyfedorov2evgenyfedorov2evgenyfedorov2 approved these changes

Assignees

@nivebb8nivebb8

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@nivebb8@evgenyfedorov2@niver2d2

[8]ページ先頭

©2009-2025 Movatter.jp