IBM drivers needs to return snapshot objects for create_cgsnapshot and delete_cgsnapshot

[Thang Pham]

This bug is a placeholder to have IBM drivers return snapshot objects instead of snapshot dictionaries for create_cgsnapshot and delete_cgsnapshot methods. It was noted on a review comment in https://review.openstack.org/#/c/163910/14//COMMIT_MSG. These drivers include:

- ibm/xiv_ds8k.py
- ibm/gpfs.py
- ibm/storwize_svc/__init__.py

This can be done be replacing occurences of db.snapshot_get_all_for_cgsnapshot() to SnapshotList.get_all_for_cgsnapshot() in create_cgsnapshot and delete_cgsnapshot. It is dependent on https://review.openstack.org/#/c/163910/.

[Sean McGinnis (sean-mcginnis)]

Automatically unassigning due to inactivity.

[OpenStack Infra (hudson-openstack)]

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

[Xinli Guan (xinli-v)]

After discussion with Ryan in Austin summit/2016, Ray wants to assign this bug to me and continue working under his guidance (mentor program).

[OpenStack Infra (hudson-openstack)]

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

commit bf2be3ddb6f9c716288c70bc7fb37c229e911788
Author: Ryan McNair <email address hidden>
Date: Fri Jan 15 18:22:30 2016 +0000

    Make IBM drivers return snapshot object for cg ops

    Update the IBM drivers to return snapshot objects (instead of dicts)
    for create_cgsnapshot and delete_cgsnapshot. Also use the snapshot
    list param passed in instead of re-querying for snapshots. Any code
    that was already being touched was fixed up to use the Snapshot/Volume
    objects (instead of dicts) as well.

    The test scenarios use objects, test_gpfs needs to use valid UUIDs
    instead of simple string such as "cg-1234" or "123456"

    Co-Authored-By: Xinli Guan <email address hidden>

    Change-Id: I826811dfdf209fab91d16dc03decfca3993500a5
    Closes-Bug: #1458128

[Doug Hellmann (doug-hellmann)]

This issue was fixed in the openstack/cinder 9.0.0.0b1 development milestone.