snapshot->provider_location not used + other fields required

[Duncan Thomas]

mysql> select * from snapshots limit 1;
+---------------------+---------------------+------------+---------+--------------------------------------+--------------------------------------+----------------------------------+----------------------------------+-----------+----------+-------------+--------------+--------------+---------------------+-------------------+
| created_at | updated_at | deleted_at | deleted | id | volume_id | user_id | project_id | status | progress | volume_size | scheduled_at | display_name | display_description | provider_location |
+---------------------+---------------------+------------+---------+--------------------------------------+--------------------------------------+----------------------------------+----------------------------------+-----------+----------+-------------+--------------+--------------+---------------------+-------------------+
| 2013-02-14 12:35:54 | 2013-02-14 12:35:54 | NULL | 0 | 6aba188b-f997-4b25-8156-237f7e4d14b3 | 4912a342-22f7-4e1e-ad9f-36627b4388e5 | 8060dc4eea694047b616348158f922cd | 3361dfd7d8bc46d0afe6010546fa0280 | available | 100% | 1 | NULL | NULL | NULL | NULL |
+---------------------+---------------------+------------+---------+--------------------------------------+--------------------------------------+----------------------------------+----------------------------------+-----------+----------+-------------+--------------+--------------+---------------------+-------------------+

The provider_location field in a cinder volume snapshot is not filled in, and provider_auth and availability zone are also needed to make the snapshot useful once its parent volume has been deleted.

(patch to fix will be submitted shortly)

[John Griffith (john-griffith)]

The provider_location entry for snapshots was added explicitly for Xen and so far hasn't been used by other folks. That being said no reason not to populate it. I'm still not convinced that deleting volumes with snapshots is necessary, between the clone option, the back-up option you're working on I'd rather see snapshot/restore staying tied to a volume but that's another topic. :)

[Sean McGinnis (sean-mcginnis)]

Automatically unassigning due to inactivity.