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

discovery: Report RSS #28931

Merged
merged 3 commits into from
Sep 2, 2024
Merged

discovery: Report RSS #28931

merged 3 commits into from
Sep 2, 2024

Conversation

vitkyrka
Copy link
Contributor

What does this PR do?

Report memory usage (RSS) for discovered services.

Motivation

https://datadoghq.atlassian.net/browse/USMON-1166

Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

Currently, the agent check uses the all process list to check if
the process is alive or not, but we can use the service list
instead, since that also considers open ports.  So if a process
closes its listening port the service is no longer alive.

This also allows to use the extra fields received from system-probe
such as RSS and allows the proc usage in the check to be eliminated
in the future.
Pass along the information about services' RSS memory received from
system-probe to the final telemetry events.
@vitkyrka vitkyrka added changelog/no-changelog team/usm The USM team qa/done QA done before merge and regressions are covered by tests labels Aug 30, 2024
@vitkyrka vitkyrka marked this pull request as ready for review August 30, 2024 14:14
@vitkyrka vitkyrka requested review from a team as code owners August 30, 2024 14:14
@vitkyrka
Copy link
Contributor Author

vitkyrka commented Sep 2, 2024

/merge

@dd-devflow
Copy link

dd-devflow bot commented Sep 2, 2024

🚂 MergeQueue: pull request added to the queue

The median merge time in main is 22m.

Use /merge -c to cancel this operation!

@dd-mergequeue dd-mergequeue bot merged commit 20d20f9 into main Sep 2, 2024
331 checks passed
@dd-mergequeue dd-mergequeue bot deleted the vincent.whitchurch/discovery-rss branch September 2, 2024 07:50
@github-actions github-actions bot added this to the 7.58.0 milestone Sep 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changelog/no-changelog component/system-probe qa/done QA done before merge and regressions are covered by tests team/usm The USM team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants