Skip to content

feat(amd-gpu): Add AMD GPU monitoring#2673

Merged
praveenkatha-nr merged 2 commits intonewrelic:releasefrom
rajrohanyadav:feat-amd-smi-gpu-monitoring
Jul 12, 2025
Merged

feat(amd-gpu): Add AMD GPU monitoring#2673
praveenkatha-nr merged 2 commits intonewrelic:releasefrom
rajrohanyadav:feat-amd-smi-gpu-monitoring

Conversation

@rajrohanyadav
Copy link
Contributor

@rajrohanyadav rajrohanyadav commented Jul 9, 2025 •
edited
Loading

Summary

Add guided install for AMD GPU instrumentation and monitoring.

Pre merge checklist

  • Did you check you NRQL syntax? - Does it work?
  • Did you include a Data source and Documentation reference?
  • Are all documentation links publicly accessible?
  • Did you check your descriptive content for voice and tone?
  • Did you check your descriptive content for spelling and grammar errors?
  • Did you review your content with a subject matter expert? (e.g. a Browser agent quickstart is reviewed with a member of the Browser Agent team)

Dashboards

  • Does the PR contain a screenshot for each of your dashboards?
  • Do your screenshots show data?
  • Has the sanitization script been run on each dashboard?

Alerts

  • Did you check that your alerts actually work?
  • Are you trying to create standalone alerts? Standalone alerts are deprecated. They should only be included in quickstarts.

@CLAassistant
Copy link

CLAassistant commented Jul 9, 2025 •
edited
Loading

CLA assistant check
All committers have signed the CLA.

@rajrohanyadav rajrohanyadav force-pushed the feat-amd-smi-gpu-monitoring branch from dbeba51 to 75eb8f8 Compare July 9, 2025 08:45
@github-actions
Copy link

github-actions bot commented Jul 9, 2025

The PR checks have run and found the following warnings:

Warning Filepath
"permissions" field should not be used dashboards/amd-gpu/amd-gpu.json

Reference the Contributing Docs for Dashboards for more information.

@rajrohanyadav rajrohanyadav force-pushed the feat-amd-smi-gpu-monitoring branch 4 times, most recently from f66e59c to 1e944aa Compare July 9, 2025 09:06
@rajrohanyadav rajrohanyadav force-pushed the feat-amd-smi-gpu-monitoring branch from 1e944aa to 3134a0b Compare July 9, 2025 09:11
@rajrohanyadav rajrohanyadav marked this pull request as ready for review July 9, 2025 09:13
@pkudikyala
Copy link
Contributor

pkudikyala commented Jul 10, 2025

Hi @rajrohanyadav ,
The documentation URL is currently unavailable. Please let me know once the docs are live, and I'll proceed with approval and merging.
https://docs.newrelic.com/docs/infrastructure/host-integrations/host-integrations-list/amd-gpu-integration/

@rajrohanyadav
Copy link
Contributor Author

rajrohanyadav commented Jul 10, 2025

Hi @rajrohanyadav , The documentation URL is currently unavailable. Please let me know once the docs are live, and I'll proceed with approval and merging. https://docs.newrelic.com/docs/infrastructure/host-integrations/host-integrations-list/amd-gpu-integration/

Yes, the PR for docs is open newrelic/docs-website#21164
Will comment, once the page is available.

pkudikyala
pkudikyala suggested changes Jul 10, 2025
View reviewed changes
Copy link
Contributor

@pkudikyala pkudikyala left a comment

Choose a reason for hiding this comment

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

Please make the necessary change.

@rajrohanyadav @pkudikyala
Update quickstarts/amd-gpu/config.yml
754769f 
Co-authored-by: pkudikyala <105910179+pkudikyala@users.noreply.github.com>
@rajrohanyadav rajrohanyadav requested a review from pkudikyala July 10, 2025 13:02
pkudikyala
pkudikyala approved these changes Jul 11, 2025
View reviewed changes
Copy link
Contributor

@pkudikyala pkudikyala left a comment

Choose a reason for hiding this comment

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

Looks good to me!

@praveenkatha-nr praveenkatha-nr merged commit 5ad1f88 into newrelic:release Jul 12, 2025
11 checks passed
@rajrohanyadav rajrohanyadav deleted the feat-amd-smi-gpu-monitoring branch July 14, 2025 06:23
praveenkatha-nr added a commit that referenced this pull request Jul 14, 2025
@praveenkatha-nr @ppant-code
@rprakash93 @rajrohanyadav @pkudikyala @nr-opensource-bot
Release 2025-07-14 (#2676)
6c42bc6 
* fix: Update eBPF dashboard and quickstart

* feat(amd-gpu): Add AMD GPU monitoring (#2673)

* feat(amd-gpu): Add AMD GPU monitoring

* Update quickstarts/amd-gpu/config.yml

Co-authored-by: pkudikyala <105910179+pkudikyala@users.noreply.github.com>

---------

Co-authored-by: pkudikyala <105910179+pkudikyala@users.noreply.github.com>

* chore: generate UUID(s) [skip ci]

* fix: Trigger build (#2675)

---------

Co-authored-by: Peeyush Pant <ppant@newrelic.com>
Co-authored-by: rprakash93 <rprakash@newrelic.com>
Co-authored-by: Rohan Yadav <rajrohanyadav@gmail.com>
Co-authored-by: pkudikyala <105910179+pkudikyala@users.noreply.github.com>
Co-authored-by: nr-opensource-bot <opensource+bot@newrelic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@pkudikyala pkudikyala pkudikyala approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@rajrohanyadav @CLAassistant @pkudikyala @praveenkatha-nr

Comments