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.

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

Diskio plugin: make devices glob also match devlinks #5815

Closed
omgold opened this issue May 7, 2019 · 3 comments
Closed

Diskio plugin: make devices glob also match devlinks #5815

omgold opened this issue May 7, 2019 · 3 comments
Labels
feature request Requests for new plugin and for new features to existing plugins
Milestone

Comments

@omgold
Copy link
Contributor

omgold commented May 7, 2019

Feature Request

Proposal:

Extend/modify the diskio plugin to to also consider aliases of device names when filtering for a subset of disk devices. I implemented the feature (roughly, without tests and such), as a basis for discussion:

omgold@267c1bc

Current behavior:

Filtering using the 'devices' options glob-matches only against actual devnode path (udev attribute DEVNAME)

Desired behavior:

Matching should also be done against components of DEVLINKS.

Use case:

I don't consider matching only the real devnodes (directly in /dev) particularly useful, as these are likely to not be consistent between reboots. Symlinks/aliases on the contrary are mostly persistent (or at least such can be created by custom udev rules).

@danielnelson
Copy link
Contributor

I think this would be a nice feature, one thing we will want to make sure of is that we only call diskInfo once. Could you open a PR?

@danielnelson danielnelson added the feature request Requests for new plugin and for new features to existing plugins label May 7, 2019
@omgold
Copy link
Contributor Author

omgold commented May 8, 2019

PR is #5817.

Remark: I didn't add any extra calls to diskInfo, as the lookup of the devlinks is added to diskName(). But there are already 2 calls, one in diskName() and one in diskTags()

@danielnelson
Copy link
Contributor

Closed in #5817

@danielnelson danielnelson added this to the 1.11.0 milestone May 15, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature request Requests for new plugin and for new features to existing plugins
Projects
None yet
Development

No branches or pull requests

2 participants