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.

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

Resource Monitoring for Windows should report container memory usage #5108

Closed
evgenyfedorov2 opened this issue Apr 16, 2024 · 2 comments · Fixed by #5301
Closed

Resource Monitoring for Windows should report container memory usage #5108

evgenyfedorov2 opened this issue Apr 16, 2024 · 2 comments · Fixed by #5301
Assignees
Labels
area-fundamentals bug This issue describes a behavior which is not expected - a bug.

Comments

@evgenyfedorov2
Copy link
Contributor

Description

Resource Monitoring for Windows reports memory usage of the process the app is running in. Instead of that, it should consider memory usage of all the processes running in the container as there might be more than one process. Resource Monitoring for Linux does the right thing already - it reports container memory usage.

Reproduction Steps

Check out this line - Resource Monitoring for Windows uses process info to report memory usage.

Expected behavior

Resource Monitoring for Windows sums up memory usage of all processes running in the same container and reports the result for the memory usage counters.

Actual behavior

Resource Monitoring for Windows reports memory usage of the process the app is running in.
Resource Monitoring for Windows reports memory usage of the whole container, e.g. all processes running in the container.

Regression?

No response

Known Workarounds

No response

Configuration

No response

Other information

No response

@evgenyfedorov2 evgenyfedorov2 added untriaged area-fundamentals bug This issue describes a behavior which is not expected - a bug. labels Apr 16, 2024
@xakep139
Copy link
Contributor

The issue seems to be a part of #4538

@RussKie
Copy link
Member

RussKie commented Apr 18, 2024

Should #4538 be closed as superceded as the Linux side is now resolved?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-fundamentals bug This issue describes a behavior which is not expected - a bug.
Projects
None yet
4 participants