Consistent Snapshots are broken in the NetApp cDOT driver

Bug #1659023 reported by Goutham Pacha Ravi on 2017-01-24
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
OpenStack Shared File Systems Service (Manila)
Medium
Ben Swartzlander

Bug Description

NetApp driver supported consistency groups since their introduction in Liberty. However, in a recent change [1], consistency groups were replaced by Share groups. This change affected the cDOT driver's ability to take group snapshots that are consistent.

[1] https://review.openstack.org/#/c/335093

description: updated
Changed in manila:
importance: Undecided → High
assignee: nobody → Goutham Pacha Ravi (gouthamr)
Changed in manila:
assignee: Goutham Pacha Ravi (gouthamr) → Ben Swartzlander (bswartz)
milestone: none → ocata-rc1
status: New → Triaged
importance: High → Medium
Changed in manila:
milestone: ocata-rc1 → pike-1
Changed in manila:
milestone: pike-1 → pike-rc1

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

Changed in manila:
status: Triaged → In Progress

Reviewed: https://review.openstack.org/491877
Committed: https://git.openstack.org/cgit/openstack/manila/commit/?id=80fe5f1fe4eb2f9d63c36fbfed3dcc388e4a931b
Submitter: Jenkins
Branch: master

commit 80fe5f1fe4eb2f9d63c36fbfed3dcc388e4a931b
Author: Ben Swartzlander <email address hidden>
Date: Thu Aug 3 16:32:07 2017 -0400

    Re-enable broken CG code in NetApp driver

    Override the new create_share_group_snapshot() method in the driver class
    and call the old CG snapshot code if the share group specifies CG
    support, otherwise fall back on the new (existing) code.

    This patch also removes dead code from the old CG feature from Newton and
    earlier releases.

    Change-Id: Ief71b9900c2c84e0df1d12d303517fa20ff7908b
    Closes-bug: #1659023

Changed in manila:
status: In Progress → Fix Released

This issue was fixed in the openstack/manila 5.0.0.0rc1 release candidate.

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

Other bug subscribers