Skip to content
This repository has been archived by the owner on Jan 20, 2018. It is now read-only.

Added support for 'SLOT ###' disk references #12

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

gregoryo2014
Copy link

In addition to 'Slot ##' and 'DISK##'.

On our (IBM xSeries) hardware, we have e.g.
/sys/class/enclosure/0:0:8:0/SLOT 000/device/block/sda

@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling 88224b7 on gregoryo2014:master into b4bc497 on spjmurray:master.

@gregoryo2014
Copy link
Author

gregoryo2014 commented Oct 9, 2017

Travis CI errors due to missing openstack api keys - same as errors on #11

@spjmurray
Copy link
Owner

Spec tests!! ❤️ Yeah Travis only allows my user to decrypt the keys hence they fail, can't have some naughty code echoing out my details :) I'll take a look soon. I really wish the driver devs would standardize their sysfs implementations...

@gregoryo2014
Copy link
Author

Do you mean the Travis keys issue gets in the way of spec tests? I'm fairly new to this setup - only ever submitted a couple of PRs.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
3 participants