ZFSSA iSCSI driver doesn't honor volume_backend_name config option.

Bug #1373621 reported by Juan Zuluaga
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Cinder
Fix Released
Medium
Juan Zuluaga

Bug Description

Unable to create ZFSSA iSCSI volumes when volume_backend_name is specified on extra-specs.
For instance:
$cinder type-create zfssa
$cinder type-key zfssa set volume_backend_name=zfssa
$ cinder extra-specs-list
+--------------------------------------+-------------+-----------------------------------------------+
| ID | Name | extra_specs |
+--------------------------------------+-------------+-----------------------------------------------+
| 3b476688-78ec-4c2b-ad90-6e7fea630a41 | zfssa | {u'volume_backend_name': u'zfssa'} |
| bcb3c21d-8f38-4f4f-a2dd-dc2d221026ce | lvmdriver-1 | {u'volume_backend_name': u'lvmdriver-1'} |
+--------------------------------------+-------------+-----------------------------------------------+

$ cinder create --volume_type zfssa --display_name jz1 1
+---------------------+--------------------------------------+
| Property | Value |
+---------------------+--------------------------------------+
| attachments | [] |
| availability_zone | nova |
| bootable | false |
| created_at | 2014-09-24T20:52:09.126973 |
| display_description | None |
| display_name | jz1 |
| encrypted | False |
| id | fcc8d621-ab12-4be0-9080-e22c4edfd531 |
| metadata | {} |
| size | 1 |
| snapshot_id | None |
| source_volid | None |
| status | creating |
| volume_type | zfssa |
+---------------------+--------------------------------------+
devstack@cam-cyclops:~$ cinder list
+--------------------------------------+--------+--------------+------+-------------+----------+-------------+
| ID | Status | Display Name | Size | Volume Type | Bootable | Attached to |
+--------------------------------------+--------+--------------+------+-------------+----------+-------------+
| fcc8d621-ab12-4be0-9080-e22c4edfd531 | error | jz1 | 1 | zfssa | false | |
+--------------------------------------+--------+--------------+------+-------------+----------+-------------+

Tags: zfssa
Changed in cinder:
assignee: nobody → Juan Zuluaga (juan-c-zuluaga)
status: New → In Progress
Revision history for this message
Juan Zuluaga (juan-c-zuluaga) wrote :

From the logs, (c-sch) , I see the following:
2014-09-24 16:52:09.309 WARNING cinder.scheduler.filter_scheduler [req-810a01f9-b28e-47a4-8988-9eb42cf61849 ae6a169853f54a8e9f631fb7b3644ea6 0942b384fc2b49f3a
f371b747983af7f] No weighed hosts found for volume with properties: {u'name': u'zfssa', u'qos_specs_id': None, u'deleted': False, u'created_at': u'2014-09-24T
20:04:04.000000', u'updated_at': None, u'extra_specs': {u'volume_backend_name': u'zfssa'}, u'deleted_at': None, u'id': u'3b476688-78ec-4c2b-ad90-6e7fea630a41'
}
2014-09-24 16:52:09.311 ERROR cinder.scheduler.flows.create_volume [req-810a01f9-b28e-47a4-8988-9eb42cf61849 ae6a169853f54a8e9f631fb7b3644ea6 0942b384fc2b49f3
af371b747983af7f] Failed to run task cinder.scheduler.flows.create_volume.ScheduleCreateVolumeTask;volume:create: No valid host was found. No weighed hosts av
ailable
.....
2014-09-24 16:57:16.149 DEBUG cinder.scheduler.host_manager [req-bc08ffc6-368d-4bf6-a129-7451d135da09 None None] Received volume service update from cam-cyclops@ZFSSAISCSIDriver: {u'QoS_support': False, u'volume_backend_name': u'ZFSSAISCSIDriver', u'free_capacity_gb': 20415, u'driver_version': u'1.0.0', u'total_capacity_gb': 20416, u'reserved_percentage': 0, u'vendor_name': u'Oracle', u'storage_protocol': u'iSCSI'} from (pid=24056) update_service_capabilities /opt/stack/cinder/cinder/scheduler/host_manager.py:434

So basically the volume_backend_name is 'hardwired' to the class name.

summary: - ZFSSA iSCSI driver not working when volume_backend_name is specified in
- extra specs
+ ZFSSA iSCSI driver volume create not working when volume_backend_name is
+ specified in extra specs
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/123882

Revision history for this message
Huang Zhiteng (zhiteng-huang) wrote : Re: ZFSSA iSCSI driver volume create not working when volume_backend_name is specified in extra specs

I'd rather change the description of this bug to "ZFS driver doesn't honor volume_backend_name config option" to be more accurate.

summary: - ZFSSA iSCSI driver volume create not working when volume_backend_name is
- specified in extra specs
+ ZFSSA iSCSI driver doesn't honor volume_backend_name config option.
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to cinder (master)

Reviewed: https://review.openstack.org/123882
Committed: https://git.openstack.org/cgit/openstack/cinder/commit/?id=5c61d57d3693523e9cbf11bf0b5b09bafe699247
Submitter: Jenkins
Branch: master

commit 5c61d57d3693523e9cbf11bf0b5b09bafe699247
Author: Juan Zuluaga <email address hidden>
Date: Wed Sep 24 18:51:07 2014 -0400

    ZFSSA iSCSI vol create fails with vol type option

    Vol create with volume-type option is not working since
    volume_backend_name contains the class name as
    predefined string. No matter what was specified in cinder.conf
    as volume_backend_name, volume creation failed.
    Multi-backend option and using extra specs to create custom volumes
    won't work.
    The fix is to look whether volume_backend_name is part of the
    configuration or falls into the class name in case there is
    no backend name.

    Change-Id: I1bc501dd4c5689d96c7beb720b64112df1770232
    Closes-Bug: 1373621
    DocImpact

Changed in cinder:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in cinder:
milestone: none → juno-rc2
Thierry Carrez (ttx)
Changed in cinder:
importance: Undecided → Medium
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to cinder (proposed/juno)

Fix proposed to branch: proposed/juno
Review: https://review.openstack.org/126864

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to cinder (proposed/juno)

Reviewed: https://review.openstack.org/126864
Committed: https://git.openstack.org/cgit/openstack/cinder/commit/?id=a8cec39f8243fd4ee6c0a16fc0620d4b0980c749
Submitter: Jenkins
Branch: proposed/juno

commit a8cec39f8243fd4ee6c0a16fc0620d4b0980c749
Author: Juan Zuluaga <email address hidden>
Date: Wed Sep 24 18:51:07 2014 -0400

    ZFSSA iSCSI vol create fails with vol type option

    Vol create with volume-type option is not working since
    volume_backend_name contains the class name as
    predefined string. No matter what was specified in cinder.conf
    as volume_backend_name, volume creation failed.
    Multi-backend option and using extra specs to create custom volumes
    won't work.
    The fix is to look whether volume_backend_name is part of the
    configuration or falls into the class name in case there is
    no backend name.

    Closes-Bug: 1373621
    DocImpact

    (cherry picked from commit 5c61d57d3693523e9cbf11bf0b5b09bafe699247)

    Change-Id: I1bc501dd4c5689d96c7beb720b64112df1770232

Changed in cinder:
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in cinder:
milestone: juno-rc2 → 2014.2
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/128920

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on cinder (master)

Change abandoned by Mike Perez (<email address hidden>) on branch: master
Review: https://review.openstack.org/128920

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to cinder (master)
Download full text (11.8 KiB)

Reviewed: https://review.openstack.org/128920
Committed: https://git.openstack.org/cgit/openstack/cinder/commit/?id=66494f54112fdfa135b3974c75aa388c8d1fb49e
Submitter: Jenkins
Branch: master

commit be3d4604dc0566e0838959d998ff1d37755de6d3
Author: Tomoki Sekiyama <email address hidden>
Date: Tue Oct 14 19:09:44 2014 -0400

    Fix LVM iSCSI driver tgtadm CHAP authentication

    Currently CHAP Authentication in LVM iSCSI driver with tgtadm does not work.
    This is because the tgtadm helper creates the target configuration file
    with an 'IncomingUser' entry, which is ignored by tgtd.
    This patch fixes it to 'incominguser'.

    Change-Id: I14871985a2a916834122f849238f05b75726bc1a
    Closes-Bug: #1329214
    (cherry picked from commit e3563891545c801726d227f752cf99488ed5c7dd)

commit f7ee62cc58d8b642af67510a310f6259492a4508
Author: Mitsuhiro Tanino <email address hidden>
Date: Tue Oct 14 12:41:41 2014 -0400

    Export cinder volumes only if the status is 'in-use'

    Currently, cinder volumes are exported both 'in-use' and 'available'
    after restarting cinder-volume service.
    This behavior was introduced following commit.

      commit ffefe18334a9456250e1b6ff88b7b47fb366f374
      Author: Zhiteng Huang <email address hidden>
      Date: Sat Aug 23 18:32:57 2014 +0000

    If the volumes are attached to nova instances, they should be exported
    via tgtd after restarting cinder-volume.
    But the volumes which are not attached to instances must not be exported
    because everyone can connect these volumes.

    This patch changes volume export behavior that exports a volume only if
    the volume status is 'in-use'.

    Change-Id: I4c598c240b9290c81bd8001e5a0720c8c329aeb9
    Signed-off-by: Mitsuhiro Tanino <email address hidden>
    Closes-bug: #1381106
    (cherry picked from commit e2f28b967910625432be0eab6a851adf53ac58ea)

commit 01e7c516852e53df661b2eedc970c327c1ff10ce
Author: Vipin Balachandran <email address hidden>
Date: Fri Oct 10 23:06:27 2014 +0530

    Revert "Relocate volume to compliant datastore"

    Commit 4be8913520f5e9fe4109ade101da9509e4a83360 introduced a regression
    which causes failures during cinder volume re-attach. This patch reverts
    commit 4be8913520f5e9fe4109ade101da9509e4a83360 as an immediate fix.

    Closes-Bug: #1379830
    Change-Id: I5dfbd45533489c3c81db8d256bbfd2f85614a357
    (cherry picked from commit 48cb82971e0418f9a629e2b39d0433dc2c0e6919)

commit 900d49723f65e87658381ff955559f54ac98c487
Author: Andreas Jaeger <email address hidden>
Date: Thu Oct 9 12:25:28 2014 +0200

    Updated translations

    Commands run:-
    $ python setup.py extract_messages
    $ python setup.py update_catalog --no-fuzzy-matching \
      --ignore-obsolete=true
    $ source \
      ../openstack-infra/project-config/jenkins/scripts/common_translation_update.sh
    $ setup_loglevel_vars
    $ cleanup_po_files cinder

    Change-Id: I73f3bdccb4be98df95fa853864e465f4d83a8884

commit 8e94aaa2b28b491314fe8642061ac73e3fe8e966
Author: Navneet Singh <email address hidden>
Date: Thu Aug 28 16:03:41 2014 +0530

    NetApp fix eseries unit test mock clean

 ...

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.