Skip to content

Add device-mapper-specific block device properties#412

Merged
SuperQ merged 8 commits intoprometheus:masterfrom
ventifus:blockdevice-stats-devicemapper
Jan 10, 2022
Merged

Add device-mapper-specific block device properties#412
SuperQ merged 8 commits intoprometheus:masterfrom
ventifus:blockdevice-stats-devicemapper

Conversation

@ventifus
Copy link
Copy Markdown
Contributor

Expose block device-mapper properties. This will enable node_exporter to expose the device-mapper information in a node_disk_info metric. (PR on that to follow.)
Closes #211.

Signed-off-by: W. Andrew Denton [email protected]

@ventifus ventifus changed the title Blockdevice stats devicemapper Add device-mapper stats Sep 14, 2021
@ventifus ventifus changed the title Add device-mapper stats Add device-mapper-specific block device properties Sep 14, 2021
@SuperQ
Copy link
Copy Markdown
Member

SuperQ commented Oct 31, 2021

This needs a rebase to update the fixtures file.

W. Andrew Denton added 4 commits November 19, 2021 11:18
@ventifus ventifus force-pushed the blockdevice-stats-devicemapper branch from ac393dc to 2129811 Compare November 19, 2021 19:19
W. Andrew Denton added 3 commits November 19, 2021 11:21
Copy link
Copy Markdown
Member

@SuperQ SuperQ left a comment

Choose a reason for hiding this comment

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

Minor nits, otherwise LGTM.

Signed-off-by: W. Andrew Denton <[email protected]>
@ventifus
Copy link
Copy Markdown
Contributor Author

ventifus commented Dec 4, 2021

Alright, I've capitalized all of the "Files with" comments (there were a few existing ones too).

@discordianfish discordianfish requested a review from SuperQ December 6, 2021 10:45
@discordianfish
Copy link
Copy Markdown
Member

@SuperQ PTAL

@ddpolyakov
Copy link
Copy Markdown

Hi there. Waiting for this feature a lot in a new build. @SuperQ could you please take a look?

Copy link
Copy Markdown
Member

@SuperQ SuperQ 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!

@SuperQ SuperQ merged commit fbe360a into prometheus:master Jan 10, 2022
remijouannet pushed a commit to remijouannet/procfs that referenced this pull request Oct 20, 2022
* Expose device-mapper information as DeviceMapperInfo

Signed-off-by: W. Andrew Denton <[email protected]>

* Enumerate block device slaves as UnderlyingDeviceInfo

Signed-off-by: W. Andrew Denton <[email protected]>

* Add missing fixtures.

Signed-off-by: W. Andrew Denton <[email protected]>

* blockdevice/stats: uppercase initialisms.

Signed-off-by: W. Andrew Denton <[email protected]>

* Expose device-mapper information as DeviceMapperInfo

Signed-off-by: W. Andrew Denton <[email protected]>

* Expose device-mapper information as DeviceMapperInfo

Signed-off-by: W. Andrew Denton <[email protected]>

* Update fixtures.

Signed-off-by: W. Andrew Denton <[email protected]>

* Capitalize comments.

Signed-off-by: W. Andrew Denton <[email protected]>

Co-authored-by: W. Andrew Denton <[email protected]>
jritter pushed a commit to jritter/procfs that referenced this pull request Jul 15, 2024
* Expose device-mapper information as DeviceMapperInfo

Signed-off-by: W. Andrew Denton <[email protected]>

* Enumerate block device slaves as UnderlyingDeviceInfo

Signed-off-by: W. Andrew Denton <[email protected]>

* Add missing fixtures.

Signed-off-by: W. Andrew Denton <[email protected]>

* blockdevice/stats: uppercase initialisms.

Signed-off-by: W. Andrew Denton <[email protected]>

* Expose device-mapper information as DeviceMapperInfo

Signed-off-by: W. Andrew Denton <[email protected]>

* Expose device-mapper information as DeviceMapperInfo

Signed-off-by: W. Andrew Denton <[email protected]>

* Update fixtures.

Signed-off-by: W. Andrew Denton <[email protected]>

* Capitalize comments.

Signed-off-by: W. Andrew Denton <[email protected]>

Co-authored-by: W. Andrew Denton <[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.

Add parsing of Device Mapper Slave Devices

4 participants