Cinder migration 017_add_encryption_information adds two new volume types without warning

Bug #1215962 reported by Michael Kerrin on 2013-08-23
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Cinder
Undecided
Duncan Thomas

Bug Description

After deploying a recent version of cinder to a test system it was noticed that the cinder migration created two new volume types without warning. It shouldn't have done that.

$ cinder type-list
+--------------------------------------+----------+
| ID | Name |
+--------------------------------------+----------+
| cdc8ac25-4d65-4488-911a-3a9b0c479bf7 | LUKS |
| d1fb688c-cf6a-4b0c-82e0-224d1ad60122 | dm-crypt |
+--------------------------------------+----------+

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

Changed in cinder:
assignee: nobody → Michael Kerrin (michael-kerrin-w)
status: New → In Progress
Changed in cinder:
assignee: Michael Kerrin (michael-kerrin-w) → Duncan Thomas (duncan-thomas)

Reviewed: https://review.openstack.org/43482
Committed: http://github.com/openstack/cinder/commit/921ce48fb12fdeb71b10cb411ff9aca9abae03d7
Submitter: Jenkins
Branch: master

commit 921ce48fb12fdeb71b10cb411ff9aca9abae03d7
Author: Michael Kerrin <email address hidden>
Date: Fri Aug 23 14:42:46 2013 +0000

    The DB migration shouldn't populate types table

    It is up to the system adminisator to do this via an API call.

    Also fixes up a few cases of dict() being used rather than the prefered {}
    (https://github.com/openstack-dev/hacking/blob/master/HACKING.rst#dictionarieslists)

    Fixes bug: 1215962

    Change-Id: I6b2eea0c03467b157ec147292dbea54e7f99006c

Changed in cinder:
status: In Progress → Fix Committed
Thierry Carrez (ttx) on 2013-09-05
Changed in cinder:
milestone: none → havana-3
status: Fix Committed → Fix Released
Thierry Carrez (ttx) on 2013-10-17
Changed in cinder:
milestone: havana-3 → 2013.2
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers