glance-manage db metadefs commands don't use transactions
Bug #1367729 reported by
Pawel Koniszewski
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Glance |
Confirmed
|
Medium
|
Unassigned | ||
Bug Description
Current approach of loading metadata definitions to database does not use transactions. Instead it inserts data to database without transactions so if something fails inside a single file, e.g. inserting properties, then user has to manually remove all related data from database, repair the json file and call 'db load_metadefs' again.
To prevent such scenario db load_metadefs should use transactions, so if something fails then user won't care about consistency of the data in database. Also to keep consistency in data seeding script all methods should be rewritten to use sessions instead of engines.
Revision history for this message
| Pawel Koniszewski (pawel-koniszewski) wrote | #1 |
| Changed in glance: | |
| assignee: | nobody → Pawel Koniszewski (pawel-koniszewski) |
| status: | New → In Progress |
| tags: | added: metadef |
| Changed in glance: | |
| assignee: | Pawel Koniszewski (pawel-koniszewski) → Marcin Karkocha (marcin-karkocha) |
| Changed in glance: | |
| assignee: | Marcin Karkocha (marcin-karkocha) → Pawel Koniszewski (pawel-koniszewski) |
| Changed in glance: | |
| importance: | Undecided → Medium |
Revision history for this message
| OpenStack Infra (hudson-openstack) wrote Change abandoned on glance (master) | #2 |
| Changed in glance: | |
| assignee: | Pawel Koniszewski (pawel-koniszewski) → nobody |
| Changed in glance: | |
| assignee: | nobody → Alexander Bashmakov (abashmak) |
| assignee: | Alexander Bashmakov (abashmak) → nobody |
| status: | In Progress → Confirmed |
To post a comment you must log in.
Review: https:/