Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

tmds
Copy link
Member

@tmds tmds commented May 2, 2023

Fixes #83129.

The WorkingSet tests fail on Fedora 38+ because a zero working set value is observed just after the process start.

@dotnet/area-system-diagnostics-process ptal.

cc @omajid

…ocess.

The WorkingSet tests fail on Fedora 38+ because a zero working set value
is observed just after the process start.
@ghost ghost added the community-contribution Indicates that the PR has been added by a community member label May 2, 2023
@ghost
Copy link

ghost commented May 11, 2023

Tagging subscribers to this area: @dotnet/area-system-diagnostics-process
See info in area-owners.md if you want to be subscribed.

Issue Details

Fixes #83129.

The WorkingSet tests fail on Fedora 38+ because a zero working set value is observed just after the process start.

@dotnet/area-system-diagnostics-process ptal.

cc @omajid

Author: tmds
Assignees: -
Labels:

area-System.Diagnostics.Process, community-contribution

Milestone: -

Copy link
Member

@adamsitnik adamsitnik left a comment

Choose a reason for hiding this comment

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

Big thanks for making our test suite ready for the newest Linux kernel @tmds!

I left two comments, my main concern is catching only exception specific for this case.

@adamsitnik adamsitnik added this to the 8.0.0 milestone May 11, 2023
@adamsitnik adamsitnik self-assigned this May 11, 2023
@tmds
Copy link
Member Author

tmds commented May 14, 2023

@adamsitnik I've addressed your feedback, ptal.

Copy link
Member

@adamsitnik adamsitnik left a comment

Choose a reason for hiding this comment

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

LGTM, thank you @tmds!

@adamsitnik adamsitnik merged commit 27d0009 into dotnet:main May 17, 2023
@ghost ghost locked as resolved and limited conversation to collaborators Jun 16, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-System.Diagnostics.Process community-contribution Indicates that the PR has been added by a community member
Projects
None yet
Development

Successfully merging this pull request may close these issues.

ProcessTests.Test[Peak]WorkingSet[64] fail on Fedora 39
2 participants