Re: [openstack-dev] [nova] if by archived you mean, wipes out your tables completely, then sure, it works fine

2015-07-10 Thread Matt Riedemann
On 3/16/2015 8:48 AM, Attila Fazekas wrote: Hi Mike, The points was, there is no real need or real use case for archiving the db as the nova-mange does. What is the exact use case ? Auditing ? Accounting ? * Keystone allows permanent delete, if you need to do auditing probably the user

Re: [openstack-dev] [nova] if by archived you mean, wipes out your tables completely, then sure, it works fine

2015-03-16 Thread Attila Fazekas
Hi Mike, The points was, there is no real need or real use case for archiving the db as the nova-mange does. What is the exact use case ? Auditing ? Accounting ? * Keystone allows permanent delete, if you need to do auditing probably the user accounts would the primary target for saving. *

Re: [openstack-dev] [nova] if by archived you mean, wipes out your tables completely, then sure, it works fine

2015-03-13 Thread Chris Friesen
On 03/12/2015 05:29 PM, Mike Bayer wrote: snip If we assume that all of our tables are filled up with zeroes for those deleted columns, because that’s the default, this **wipes the whole table clean**. How do the tests pass? Well the tests are in test_db_api-ArchiveTestCase, and actually,

Re: [openstack-dev] [nova] if by archived you mean, wipes out your tables completely, then sure, it works fine

2015-03-13 Thread Thomas Herve
[snip] If we assume that all of our tables are filled up with zeroes for those deleted columns, because that’s the default, this **wipes the whole table clean**. How do the tests pass? Well the tests are in test_db_api-ArchiveTestCase, and actually, they don’t. But they don’t fail every

Re: [openstack-dev] [nova] if by archived you mean, wipes out your tables completely, then sure, it works fine

2015-03-13 Thread Attila Fazekas
The archiving has issues since very long time [1], something like this [2] is expected to replace it. The archiving just move trash to the other side of the desk, usually just permanently deleting everything what is deleted for more than 7 day is better for everyone. For now, maybe just wiping

Re: [openstack-dev] [nova] if by archived you mean, wipes out your tables completely, then sure, it works fine

2015-03-13 Thread Sean Dague
On 03/13/2015 07:07 AM, Sean Dague wrote: On 03/13/2015 05:54 AM, Thomas Herve wrote: [snip] If we assume that all of our tables are filled up with zeroes for those deleted columns, because that’s the default, this **wipes the whole table clean**. How do the tests pass? Well the tests are

Re: [openstack-dev] [nova] if by archived you mean, wipes out your tables completely, then sure, it works fine

2015-03-13 Thread Sean Dague
On 03/13/2015 05:54 AM, Thomas Herve wrote: [snip] If we assume that all of our tables are filled up with zeroes for those deleted columns, because that’s the default, this **wipes the whole table clean**. How do the tests pass? Well the tests are in test_db_api-ArchiveTestCase, and

Re: [openstack-dev] [nova] if by archived you mean, wipes out your tables completely, then sure, it works fine

2015-03-13 Thread Sean Dague
On 03/13/2015 07:57 AM, Sean Dague wrote: On 03/13/2015 07:07 AM, Sean Dague wrote: On 03/13/2015 05:54 AM, Thomas Herve wrote: [snip] If we assume that all of our tables are filled up with zeroes for those deleted columns, because that’s the default, this **wipes the whole table clean**.

Re: [openstack-dev] [nova] if by archived you mean, wipes out your tables completely, then sure, it works fine

2015-03-13 Thread Mike Bayer
Attila Fazekas afaze...@redhat.com wrote: The archiving has issues since very long time [1], something like this [2] is expected to replace it. yeah I was thinking of just rewriting the archive routine in Nova to be reasonable, but I can build this routine into Oslo.db as well as a generic

[openstack-dev] [nova] if by archived you mean, wipes out your tables completely, then sure, it works fine

2015-03-12 Thread Mike Bayer
Hello Nova - Not sure if I’m just staring at this for too long, or if archive_deleted_rows_for_table() is just not something we ever use. Because it looks like it’s really, really broken very disastrously, and I’m wondering if I’m just missing something in front of me. Let’s look at what it