[cinder] os-vol-host-attr:host is wrong when backend is NetApp NFS

Bug #1610676 reported by Serg Lystopad
12
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Mirantis OpenStack
Fix Released
High
Ivan Kolodyazhny
8.0.x
Fix Released
High
Ivan Kolodyazhny

Bug Description

Detailed bug description:
when user creates bunch of VMs using 'Boot from volume (creates a new volume)' Horizon shows that volumes have been created on different NFS shares (have been distributed almost evenly between different shares). Actually almost all volumes have been created on one of the configured shares.
So there are 2 issues in this case:
 - cinder doesn't distribute volumes between back-end shares
 - os-vol-host-attr:host property is incorrectly reported for some volumes

> for vol_id in $(cinder list --tenant 237abf0b66c84a8aa3fcf8399b84995f | sed '1,3d' | sed '$d' | sed 's/\ //g' | cut -f2 -d'|'); do vol_host=$(cinder show $vol_id | grep os-vol-host-attr:host | sed 's/\ //g' | cut -f3 -d'|'); echo "$vol_host $vol_id"; done | sort -k1 | cut -f1 -d ' ' | uniq -c
     27 str:netapp@cinder_netapp#172.18.52.240:/vol_cin2
     33 str:netapp@cinder_netapp#172.18.52.240:/vol_cin3
     30 str:netapp@cinder_netapp#172.18.52.240:/vol_cin4

I expect 30 volumes on 172.18.52.240:/vol_cin4 share, but actually only 4 are there

> mount | grep vol_cin
172.18.52.240:/vol_cin1 on /var/lib/cinder/mnt/38bc0a0c3d2f51eaae54acf7edd7a718 type nfs (rw,addr=172.18.52.240)
172.18.52.240:/vol_cin2 on /var/lib/cinder/mnt/f7be1a30ac7d2a5bf5d64a909665ca93 type nfs (rw,addr=172.18.52.240)
172.18.52.240:/vol_cin3 on /var/lib/cinder/mnt/5f779d92fd63c7ce63935cdced8b474b type nfs (rw,addr=172.18.52.240)
172.18.52.240:/vol_cin4 on /var/lib/cinder/mnt/f72992e58017c3e7f5b81fd75487eecb type nfs (rw,addr=172.18.52.240)

> for mp in $(mount | grep vol_cin | cut -f3 -d ' '); do echo "$mp $(ls $mp/volume* | wc -l)"; done
/var/lib/cinder/mnt/38bc0a0c3d2f51eaae54acf7edd7a718 382
/var/lib/cinder/mnt/f7be1a30ac7d2a5bf5d64a909665ca93 4
/var/lib/cinder/mnt/5f779d92fd63c7ce63935cdced8b474b 14
/var/lib/cinder/mnt/f72992e58017c3e7f5b81fd75487eecb 4

Steps to reproduce:
spawn 100 VMs using 'Boot from volume (creates a new volume)', check os-vol-host-attr:host property for volumes, check actual place where volume file has been created

Description of the environment:
MOS8 (liberty), fuel-plugin-cinder-netapp (4.1.1) with NetApp NFS backend (4 NFS shares are configured)

description: updated
Changed in mos:
status: New → Confirmed
importance: Undecided → High
assignee: nobody → MOS Cinder (mos-cinder)
milestone: none → 9.1
tags: added: area-cinder
Ivan Kolodyazhny (e0ne)
Changed in mos:
assignee: MOS Cinder (mos-cinder) → Ivan Kolodyazhny (e0ne)
Revision history for this message
Ivan Kolodyazhny (e0ne) wrote :
Ivan Kolodyazhny (e0ne)
Changed in mos:
status: Confirmed → In Progress
Revision history for this message
Ivan Kolodyazhny (e0ne) wrote :

Patch for MOS 8 merged: https://review.fuel-infra.org/24807

Ivan Kolodyazhny (e0ne)
Changed in mos:
status: In Progress → Fix Committed
tags: added: on-verification
Revision history for this message
Ivan Kolodyazhny (e0ne) wrote :
Revision history for this message
Sofiia Andriichenko (sandriichenko) wrote :

Unfortunately we don't have environment with NetApp, we checked that the fix is presented in the code on MOs 9.1 snapshot #251

Changed in mos:
status: Fix Committed → Fix Released
Revision history for this message
Vadim Rovachev (vrovachev) wrote :

We have not possibility to check this patch. We checked that code from patch contains in MU3 packages.

Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Fix proposed to openstack/cinder (mcp/newton)

Fix proposed to branch: mcp/newton
Change author: Ivan Kolodyazhny <email address hidden>
Review: https://review.fuel-infra.org/33419

Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Fix proposed to openstack/cinder (11.0/ocata)

Fix proposed to branch: 11.0/ocata
Change author: Ivan Kolodyazhny <email address hidden>
Review: https://review.fuel-infra.org/34086

Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Fix proposed to openstack/cinder (mcp/ocata)

Fix proposed to branch: mcp/ocata
Change author: Ivan Kolodyazhny <email address hidden>
Review: https://review.fuel-infra.org/34678

Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Change abandoned on openstack/cinder (11.0/ocata)

Change abandoned by Michael Dovgal <email address hidden> on branch: 11.0/ocata
Review: https://review.fuel-infra.org/34086
Reason: This driver was already refactored

Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Change abandoned on openstack/cinder (mcp/ocata)

Change abandoned by Michael Dovgal <email address hidden> on branch: mcp/ocata
Review: https://review.fuel-infra.org/34678
Reason: This driver was already refactored

Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Change abandoned on openstack/cinder (mcp/newton)

Change abandoned by Michael Dovgal <email address hidden> on branch: mcp/newton
Review: https://review.fuel-infra.org/33419

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.