diff options
author | Aija Jaunteva <aija.jaunteva@dell.com> | 2020-04-20 21:33:36 +0300 |
---|---|---|
committer | Aija Jaunteva <aija.jaunteva@dell.com> | 2020-04-21 18:41:00 +0000 |
commit | 722d9d45218eeaa0285907cdc12ec77968b5693a (patch) | |
tree | 142b2ecaccf08ef2e922bbe61d138843778ffa1c | |
parent | 9c116c4ef3d10a2eadb0fb0ddd13eae24a470f99 (diff) | |
download | ironic-722d9d45218eeaa0285907cdc12ec77968b5693a.tar.gz |
Fix SpanLength calculation for DRAC RAID configuration
Caused by differences in number division in Python 2 and
Python 3, so forcing type to be int instead of float
as expected by idrac.
Change-Id: I6a20ec52a8c464aaf275583fb21607fffb3b1f3b
Story: 2004265
Task: 27804
(cherry picked from commit 27a0b54b6c5d5af19a3888a5d3348cac4ba9ffb0)
-rw-r--r-- | ironic/drivers/modules/drac/raid.py | 2 | ||||
-rw-r--r-- | releasenotes/notes/bug-2004265-cd9056868295f374.yaml | 7 |
2 files changed, 8 insertions, 1 deletions
diff --git a/ironic/drivers/modules/drac/raid.py b/ironic/drivers/modules/drac/raid.py index 633216d1c..56f8cbc90 100644 --- a/ironic/drivers/modules/drac/raid.py +++ b/ironic/drivers/modules/drac/raid.py @@ -495,7 +495,7 @@ def _calculate_volume_props(logical_disk, physical_disks, free_space_mb): error_msg = _('invalid number of physical disks was provided') raise exception.DracOperationError(error=error_msg) - disks_per_span = len(selected_disks) / spans_count + disks_per_span = int(len(selected_disks) / spans_count) # Best practice is to not pass span_length and span_depth when creating a # RAID10. The iDRAC will dynamically calculate these values using maximum diff --git a/releasenotes/notes/bug-2004265-cd9056868295f374.yaml b/releasenotes/notes/bug-2004265-cd9056868295f374.yaml new file mode 100644 index 000000000..5d25b5ea8 --- /dev/null +++ b/releasenotes/notes/bug-2004265-cd9056868295f374.yaml @@ -0,0 +1,7 @@ +--- +fixes: + - | + Fixes 'Invalid parameter value for SpanLength' when configuring RAID + using Python 3. This passed incorrect data type to iDRAC, e.g., instead + of `2` it passed `2.0`. + See `story 2004265 <https://storyboard.openstack.org/#!/story/2004265>`_. |