| 2016-05-06 17:23:21 |
Alfredo Moralejo |
description |
Hi,
Some problems has been detected when working with ontap_cluster family type:
- Since liberty, netapp_eseries_host_type has been deprecatd to netapp_host_type:
https://github.com/openstack/cinder/blob/9567844f0e180a7278767fdcd4378b19913933be/cinder/volume/drivers/netapp/options.py#L171-L176
- Additionally, this parameters accepts different values depending on the family_type:
- For eseries, it defaults to linux_dm_mp
https://github.com/openstack/cinder/blob/0ca7afbe9e63c30e48123350a8b6150a35c45c34/cinder/volume/drivers/netapp/eseries/library.py#L115
- For ontap_cluster, it defaults to 'linux' and linux_dm_mp is not a valid option. values acceptable are in:
https://github.com/openstack/cinder/blob/0ca7afbe9e63c30e48123350a8b6150a35c45c34/cinder/volume/drivers/netapp/dataontap/block_base.py#L84-L86 . Current default
Currently puppet-cinder defaults to linux_dm_mp which is invalid for ontap_cluster.
My suggestion would be to default netapp_host_type (or netapp_eseries_host_type) to null (as driver default since liberty) so that it picks the per-driver default and only pass a specific valued when a parameter is passed to cinder::backend::netapp.
Best regards,
Alfredo |
Hi,
Some problems has been detected when working with ontap_cluster family type:
- Since liberty, netapp_eseries_host_type has been deprecatd to netapp_host_type:
https://github.com/openstack/cinder/blob/master/cinder/volume/drivers/netapp/options.py#L171-L176
- Additionally, this parameters accepts different values depending on the family_type:
- For eseries, it defaults to linux_dm_mp
https://github.com/openstack/cinder/blob/master/cinder/volume/drivers/netapp/eseries/library.py#L115
- For ontap_cluster, it defaults to 'linux' and linux_dm_mp is not a valid option. values acceptable are in:
https://github.com/openstack/cinder/blob/master/cinder/volume/drivers/netapp/dataontap/block_base.py#L84-L86
Currently puppet-cinder defaults to linux_dm_mp which is invalid for ontap_cluster.
My suggestion would be to default netapp_host_type (or netapp_eseries_host_type) to null (as driver default since liberty) so that it picks the per-driver default and only pass a specific valued when a parameter is passed to cinder::backend::netapp.
Best regards,
Alfredo |
|
