Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Add capture to open file descriptors in system metrics instrumentation #2652

Merged
merged 8 commits into from
Jul 5, 2024

Conversation

emdneto
Copy link
Member

@emdneto emdneto commented Jul 2, 2024

Description

Fixes #2562

Type of change

Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

How Has This Been Tested?

  • tox

Does This PR Require a Core Repo Change?

  • Yes. - Link to PR:
  • No.

Checklist:

See contributing.md for styleguide, changelog guidelines, and more.

  • Followed the style guidelines of this project
  • Changelogs have been updated
  • Unit tests have been added
  • Documentation has been updated

Copy link
Contributor

@xrmx xrmx left a comment

Choose a reason for hiding this comment

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

This requires a changelog entry

@emdneto emdneto requested a review from a team July 2, 2024 15:34
@ocelotl ocelotl force-pushed the num-fds-system-metrics branch from 1a9eed7 to 02add2f Compare July 3, 2024 23:24
@ocelotl ocelotl merged commit c4c9b6b into open-telemetry:main Jul 5, 2024
379 checks passed
xrmx pushed a commit to xrmx/opentelemetry-python-contrib that referenced this pull request Jan 24, 2025
open-telemetry#2652)

* add capture to open file descriptors in system metrics instrumentation

* fix meter type

* fix pypy test

* add changelog entry

* Update instrumentation/opentelemetry-instrumentation-system-metrics/src/opentelemetry/instrumentation/system_metrics/__init__.py

Co-authored-by: Leighton Chen <lechen@microsoft.com>

* remove runtiem prefix

* update tests

---------

Co-authored-by: Leighton Chen <lechen@microsoft.com>
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

looks like process.open_file_descriptor.count is not implemented in python
4 participants