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.

Sign up for GitHub

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

Jump to bottom

[nvml] add compute_running_process (processes running in gpu) #1580

Merged
merged 6 commits into from
Nov 3, 2022
Merged

[nvml] add compute_running_process (processes running in gpu) #1580

merged 6 commits into from
Nov 3, 2022

Conversation

BeLeap
Copy link
Contributor

@BeLeap BeLeap commented Oct 24, 2022 •
edited

What does this PR do?

Reports process pids running in GPU

Motivation

We are using Datadog to monitor processes running in GPU.

Review checklist

  • PR has a meaningful title or PR has the no-changelog label attached
  • Feature or bugfix has tests
    Test Result image
  • Git history is clean
  • If PR impacts documentation, docs team has been notified or an issue has been opened on the documentation repo

@BeLeap BeLeap requested a review from a team as a code owner October 24, 2022 02:09
@BeLeap BeLeap mentioned this pull request Oct 24, 2022
Open
@BeLeap BeLeap changed the title [nvml] add compute running processes (processes running in gpu) [nvml] add compute_running_process (processes running in gpu) Oct 24, 2022
@urseberry urseberry added the editorial review Waiting on a more in-depth review from a docs team editor label Oct 24, 2022
@urseberry
Copy link
Contributor

Docs review on DOCS-4273

@FlorentClarret
Copy link
Member

Hi! Thank you for the PR.

Pinging @cep21 as maintainer of the nvml integration.

hestonhoffman
hestonhoffman approved these changes Nov 3, 2022
View reviewed changes
Copy link
Contributor

@hestonhoffman hestonhoffman left a comment

Choose a reason for hiding this comment

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

Docs LGTM

@FlorentClarret FlorentClarret merged commit 21a3854 into DataDog:master Nov 3, 2022
liavt pushed a commit to blinkops/integrations-extras that referenced this pull request Nov 13, 2022
@BeLeap @liavt
[nvml] add compute_running_process (processes running in gpu) (Data…
07131cb 
…Dog#1580)

* add process reporting

* add metadata

* fix failing test

* update version

* fix styling issues

* sync config.yaml.example
DannyBrock pushed a commit to rajeevthefiddler/integrations-extras that referenced this pull request Dec 27, 2022
@BeLeap @DannyBrock
[nvml] add compute_running_process (processes running in gpu) (Data…
d650dbe 
…Dog#1580)

* add process reporting

* add metadata

* fix failing test

* update version

* fix styling issues

* sync config.yaml.example
DanielDinsh pushed a commit to blinkops/integrations-extras that referenced this pull request Jun 21, 2023
@BeLeap @DanielDinsh
[nvml] add compute_running_process (processes running in gpu) (Data…
7f0a0c0 
…Dog#1580)

* add process reporting

* add metadata

* fix failing test

* update version

* fix styling issues

* sync config.yaml.example
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@cep21 cep21 cep21 approved these changes

@hestonhoffman hestonhoffman hestonhoffman approved these changes

Assignees
No one assigned
Labels
editorial review Waiting on a more in-depth review from a docs team editor
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
5 participants
@BeLeap @urseberry @FlorentClarret @cep21 @hestonhoffman