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

Make disk device name selector configurable #117

Merged

Conversation

cbeneke
Copy link
Contributor

@cbeneke cbeneke commented Nov 7, 2018

If your cluster runs with different diskselectors the recording rules for disks are broken (e.g. KVM default is vd[a-z]). This PR makes the selector configurable

@metalmatze
Copy link
Member

Maybe it makes sense to do the same as the fsTypes?

// This list of filesystem is referenced in various expressions.
fstypes: ['ext[234]', 'btrfs', 'xfs', 'zfs'],
fstypeSelector: 'fstype=~"%s"' % std.join('|', self.fstypes),

@cbeneke cbeneke force-pushed the configuruable-disk-device branch from 4e1722f to 16d5f19 Compare November 8, 2018 10:55
@cbeneke
Copy link
Contributor Author

cbeneke commented Nov 8, 2018

Imho this just complicates things, but as you can still overwrite the Selector I'm fine with that^^ Updated

@Reamer
Copy link

Reamer commented Nov 26, 2018

Maybe we should also add rbd

@cbeneke
Copy link
Contributor Author

cbeneke commented Nov 26, 2018

I just used the existing common disk-identifiers as before. I guess as defaults it is useful to just use the big common providers (google, aws, azure?) and typical linux disk device names, which should be sd* (google, azure, local), xvd* (aws) and nvme* (aws ssd, local). Not sure, if there are typical other namings for those providers though :)

@Reamer
Copy link

Reamer commented Nov 26, 2018

rbd means rados-block-device. This disk values are quiet important for ceph users.
I think this should be added in a seperate commit.

This adds rbd (Rados Block Storage) and vd (KVM default devices) to
the list of default diskDevices.
@cbeneke cbeneke force-pushed the configuruable-disk-device branch from 16d5f19 to 42e0f4f Compare November 26, 2018 16:20
@cbeneke
Copy link
Contributor Author

cbeneke commented Nov 26, 2018

Yeah.. But as I argued in my answer before, I'm not sure how common Ceph disk devices actually are. Nonetheless I added a second commit adding rbd.+ and vd.+ (KVM/Openstack) namings to the list of default device names.

@metalmatze
Copy link
Member

I would almost argue that we only have a very few examples and that's it. It's quite simple to set what you exactly need. Nonetheless it has been worked on now, so I'm find with this.

@metalmatze metalmatze merged commit 64a0675 into kubernetes-monitoring:master Nov 28, 2018
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.

3 participants