vmware : profile attributes not reflecting in VC UI

Bug #1291804 reported by satyadev svn
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Cinder
Fix Released
High
Vipin Balachandran

Bug Description

1) create profile say 'default' having content tagged to nfs data store.
2) Now 'default' profile in /etc/cinder/cinder.conf for "pbm_default_policy=default"
3) now restart cinder api service.
4) Now create volume with out any volume type "cinder create --name testvol 1"
5) Now attach to instance.
6) As per default profile define respective volume is present to nfs data store.

how ever default profile details not appearing in " VM storage policies pane" . See attached snapshot for reference

ssatya@devstack:/etc/cinder$ cat cinder.conf | grep pbm
pbm_default_policy=default

ssatya@devstack:/etc/cinder$ cinder list | grep in-use
| 4db6af03-7c60-469d-803e-2ba21893fc0d | in-use | testvol2 | 1 | None | false | 26ffc3d5-af49-4c04-926f-9b94563bfd65 |
| 862219d5-941a-4f96-85a8-9b73a83d4507 | in-use | testvol3 | 1 | None | false | 26ffc3d5-af49-4c04-926f-9b94563bfd65 |
| d33d102f-f863-441d-bf87-29465cef3760 | in-use | ThickSliver_vol1 | 1 | ThickSliver_volume | false | 26ffc3d5-af49-4c04-926f-9b94563bfd65 |

Tags: vmdk vmware
Revision history for this message
satyadev svn (svnsatya) wrote :
Revision history for this message
Huang Zhiteng (zhiteng-huang) wrote :

It seems to me that the 'pbm_default_policy' configure option only means something to VMDK driver, which means you should restart cinder volume service instead of cinder api. Could you please verify if that works for you?

Revision history for this message
satyadev svn (svnsatya) wrote :

as you suggested I restarted c-api & c-vol services. Still in VC UI " VM storage policies pane" is empty.

Revision history for this message
Vipin Balachandran (vbala) wrote :

---
        type_id = volume['volume_type_id']
        if not type_id:
            return
---

The problem is, if no volume_type specified, the profile defaults to None instead of pbm_default_policy.
The fix for Bug 1291181 removes pbm_default_policy. After that fix, the profile is queried using the storage_profile extra_spec property in the volume_type. The user can set a profile in the default_volume_type instead of pbm_default_policy.

Changed in cinder:
milestone: none → icehouse-rc1
assignee: nobody → Vipin Balachandran (vbala)
importance: Undecided → High
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to cinder (master)

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

Changed in cinder:
status: New → In Progress
Changed in cinder:
milestone: icehouse-rc1 → next
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to cinder (master)

Reviewed: https://review.openstack.org/81044
Committed: https://git.openstack.org/cgit/openstack/cinder/commit/?id=1e4d070a5b638de58c87363b18a76171dcd0ce38
Submitter: Jenkins
Branch: master

commit 1e4d070a5b638de58c87363b18a76171dcd0ce38
Author: Vipin Balachandran <email address hidden>
Date: Mon Mar 17 23:46:22 2014 +0530

    vmware: Remove pbm_default_policy config option

    The pbm_default_policy config option is used to associate a storage
    policy with the volume backing if there is no associated volume_type
    or the volume_type doesn't have the storage_profile option. A better
    approach is to set the storage_profile option in the default_volume_type.
    This change removes the pbm_default_policy and queries the policy
    using the storage_profile option in the given volume_type.

    Closes-Bug: #1291181
    Closes-Bug: #1291804
    DocImpact

    Change-Id: I2519ec1145712ce6927dacde91db468d066af021

Changed in cinder:
status: In Progress → Fix Committed
satyadev svn (svnsatya)
Changed in cinder:
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in cinder:
milestone: next → none
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.