NetApp E-Series Drivers reporting incorrect provisioned_capacity_gb

[Goutham Pacha Ravi]

This bug was split from https://bugs.launchpad.net/cinder/+bug/1714209 to show separation of drivers belonging to NetApp.

The NetApp E-Series driver is incorrectly reporting "provisioned_capacity_gb", probably because provisioned capacity means something different on the NetApp E-Series storage arrays.

Reported values are always actual physical used size [1], which is not correct as it should contain the maximum size that all volumes present in the backend could reach [2], total virtual size if you will.

Due to this over provisioning doesn't work as expected. A clear example is if you use thin provisioning with a 1.0 max_over_provisioning ratio, as you'll still be doing over provisioning.

[1] https://github.com/openstack/cinder/blob/55e29347221f1c3c1fca26ab0144ecc1f54b87b6/cinder/volume/drivers/netapp/eseries/library.py#L1466
[2] https://review.openstack.org/#/c/490116/2/specs/queens/provisioning-improvements.rst@103

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: master
Review: https://review.openstack.org/518406

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/518406
Committed: https://git.openstack.org/cgit/openstack/cinder/commit/?id=f905253b9443db1870c3d2b7b70e032bb089efa0
Submitter: Zuul
Branch: master

commit f905253b9443db1870c3d2b7b70e032bb089efa0
Author: Adriano Rosso <email address hidden>
Date: Fri Sep 29 10:29:48 2017 -0300

    NetApp E-series: Fix provisioned_capacity_gb

    Currently, NetApp E-series drivers (iSCSI and FC) are incorrectly
    reporting the total physical used size as the provisioned_capacity_gb.
    This patch fixes the calculation of this attribute by summing the
    user-visible capacity of all the volumes present in the backend and
    reporting the result as the provisioning_capacity_gb.

    Change-Id: I1471ecfffe340dafb1c95bac5eaff79cde650955
    Closes-Bug: #1718739

[OpenStack Infra (hudson-openstack)]

This issue was fixed in the openstack/cinder 12.0.0.0b2 development milestone.

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: stable/pike
Review: https://review.openstack.org/528783

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/528783
Committed: https://git.openstack.org/cgit/openstack/cinder/commit/?id=e2b755590a952dbdd490d0d02f53cf6852463cce
Submitter: Zuul
Branch: stable/pike

commit e2b755590a952dbdd490d0d02f53cf6852463cce
Author: Adriano Rosso <email address hidden>
Date: Fri Sep 29 10:29:48 2017 -0300

    NetApp E-series: Fix provisioned_capacity_gb

    Currently, NetApp E-series drivers (iSCSI and FC) are incorrectly
    reporting the total physical used size as the provisioned_capacity_gb.
    This patch fixes the calculation of this attribute by summing the
    user-visible capacity of all the volumes present in the backend and
    reporting the result as the provisioning_capacity_gb.

    (cherry picked from commit f905253b9443db1870c3d2b7b70e032bb089efa0)
    Change-Id: I1471ecfffe340dafb1c95bac5eaff79cde650955
    Closes-Bug: #1718739

[OpenStack Infra (hudson-openstack)]

This issue was fixed in the openstack/cinder 11.1.1 release.