Skip to content

Add working-set and page-fault metrics #666

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

Merged
merged 1 commit into from
Aug 10, 2020
Merged

Add working-set and page-fault metrics #666

merged 1 commit into from
Aug 10, 2020

Conversation

pmcollins
Copy link
Member

Description: Adds memory working set, page faults, and major page faults metrics to kubeletstats receiver.

Testing: Unit tests added using test fixtures recorded from minikube.

@codecov
Copy link

codecov bot commented Aug 7, 2020

Codecov Report

Merging #666 into master will increase coverage by 0.01%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #666      +/-   ##
==========================================
+ Coverage   86.61%   86.62%   +0.01%     
==========================================
  Files         197      197              
  Lines       10748    10757       +9     
==========================================
+ Hits         9309     9318       +9     
  Misses       1110     1110              
  Partials      329      329              
Flag Coverage Δ
#integration 71.09% <ø> (ø)
#unit 86.46% <100.00%> (+0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
receiver/kubeletstatsreceiver/kubelet/mem.go 100.00% <100.00%> (ø)
receiver/kubeletstatsreceiver/kubelet/metrics.go 100.00% <100.00%> (ø)

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 d632670...9c7135b. Read the comment docs.

@pmcollins pmcollins marked this pull request as ready for review August 7, 2020 22:18
@pmcollins pmcollins requested a review from a team August 7, 2020 22:18
@dmitryax
Copy link
Member

dmitryax commented Aug 8, 2020

Could you please add signalfx translations after to:

container_memory_major_page_faults
container_memory_page_faults
container_memory_working_set_bytes

It can be done as a separate PR

@dmitryax
Copy link
Member

dmitryax commented Aug 8, 2020

Please add kubeletstats receiver to the commit message

@pmcollins
Copy link
Member Author

Please add kubeletstats receiver to the commit message

Done.

@pmcollins
Copy link
Member Author

Could you please add signalfx translations after to:

container_memory_major_page_faults
container_memory_page_faults
container_memory_working_set_bytes

It can be done as a separate PR

Separate PR: #679

@bogdandrutu bogdandrutu merged commit 970b16d into open-telemetry:master Aug 10, 2020
@pmcollins pmcollins deleted the ks-mem-metrics branch August 21, 2020 16:02
ljmsc referenced this pull request in ljmsc/opentelemetry-collector-contrib Feb 21, 2022
* Bump golangci-lint to 1.25

* Skip the "unused" linter when running golangci-lint in --fix mode

This linter seems to provide bogus fix-up information to
golangci-lint, which results in file corruption instead of fixing the
issue. Since we are going to run the linter again anyway, but without
the --fix mode, skip the "unused" linter explicitly for the first run
- it will still have a chance to report errors on the second run.
codeboten pushed a commit that referenced this pull request Nov 23, 2022
The CI is not able to catch many documentation problems because we are ignoring
warnings. This commit fixes most of the warnings, ignores the rest and enables
a flag to treat them as errors.

Co-authored-by: Diego Hurtado <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants