Skip to content

Conversation

jritter
Copy link
Contributor

@jritter jritter commented Jul 15, 2024

This Pull Request adds functionality to read DRM information from the DRM kernel subsystem. This can be useful to get information of GPUs and their ports. As per now, it is possible to get the GPUs with the driver name, and for each of their ports the connection state, enablement state and DPMS state. In the future, it could be enhanced with EDID information, and more GPU driver specific information.


// DrmCard contains info from files in /sys/class/drm for a
// single DRM Card device.
type DrmCard struct {
Copy link
Member

Choose a reason for hiding this comment

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

Drm -> DRM as of golang naming standard

type DrmCardPort struct {
Name string
Status string
Dpms string
Copy link
Member

Choose a reason for hiding this comment

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

Same -> DPMS

@@ -0,0 +1,128 @@
// Copyright 2018 The Prometheus Authors
Copy link
Member

Choose a reason for hiding this comment

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

Update Year (same below)

@jritter
Copy link
Contributor Author

jritter commented Jul 23, 2024

Hi @discordianfish, thanks for your feedback. I've integrated your suggestions into the PR.

Copy link
Member

@discordianfish discordianfish left a comment

Choose a reason for hiding this comment

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

LGTM, thanks! And sorry for the delay

@jritter
Copy link
Contributor Author

jritter commented Nov 4, 2024

Hi, is there a reason why this is not being merged?

@jritter
Copy link
Contributor Author

jritter commented Dec 6, 2024

Hi @SuperQ , is there anything I can do to get this PR merged?

@jritter
Copy link
Contributor Author

jritter commented Jan 27, 2025

Hi, is there an update on this? Would be great to have this merged. Thanks for a feedback.

@discordianfish discordianfish merged commit 6185448 into prometheus:master Feb 2, 2025
8 checks passed
@discordianfish
Copy link
Member

Thanks for your contribution and sorry again for the delay(s)

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.

2 participants