Skip to content
This repository has been archived by the owner on Nov 29, 2021. It is now read-only.

Get memory usage #207

Merged
merged 6 commits into from
Feb 21, 2020
Merged

Get memory usage #207

merged 6 commits into from
Feb 21, 2020

Conversation

bjoernricks
Copy link
Contributor

Add a get_memory_usage command. Currently it is only possible to list the memory usage of the main process and its direct children.

Copy link
Member

@jjnicola jjnicola left a comment

Choose a reason for hiding this comment

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

It looks good. Could you update the documentation about this new OSP command. The changelog entry should be added too.

The command allows to return the current memory usage of the main
process and its subprocesses. Currently it only can list the current
process of the request handling process and its direct children.
Therefore it show the main process and the target processes only.
@codecov
Copy link

codecov bot commented Feb 21, 2020

Codecov Report

Merging #207 into master will decrease coverage by 0.06%.
The diff coverage is 28.94%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #207      +/-   ##
==========================================
- Coverage   73.36%   73.29%   -0.07%     
==========================================
  Files          18       18              
  Lines        2181     2191      +10     
==========================================
+ Hits         1600     1606       +6     
- Misses        581      585       +4
Impacted Files Coverage Δ
ospd/misc.py 51.56% <ø> (+4.98%) ⬆️
ospd/main.py 0% <0%> (ø) ⬆️
ospd/protocol.py 95% <100%> (+7.5%) ⬆️
ospd/ospd.py 72.72% <27.27%> (-0.59%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b9aac16...839c6c2. Read the comment docs.

@jjnicola jjnicola merged commit edc3ca0 into greenbone:master Feb 21, 2020
@bjoernricks bjoernricks deleted the get-memory-usage branch February 21, 2020 14:44
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants