Comment 5 for bug 1364279

Revision history for this message
John Griffith (john-griffith) wrote :

I'd agree we need to fix this up, especially in the case of items like LVM that don't even use Pools. Two points from my opinion:
1. We should be able to just ignore pool designation for devices that don't have/support pools
2. We most certainly should be able to give the admin a method for discovering pools

This is where my concerns regarding late merge of large features like this come into play. I think the feature impl that was done here is great work, but given the timing of it's merge we now have to scramble for all the vendors to implement and start finding the corner cases that we missed in the original implementation. Not a fun place to be IMO.